Skip to content

Add CLI trace directory support#25

Merged
ultmaster merged 1 commit intomainfrom
gzm40j-codex/add-trace-dump-for-poml-calls
Jun 24, 2025
Merged

Add CLI trace directory support#25
ultmaster merged 1 commit intomainfrom
gzm40j-codex/add-trace-dump-for-poml-calls

Conversation

@ultmaster
Copy link
Contributor

Summary

  • keep Node.js tracing robust by collecting errors when traces fail
  • forward trace directory via --traceDir from the Python wrapper
  • avoid modifying POML_TRACE env var from library code
  • address code review feedback on trace dumping logic

Testing

  • npm run build-cli
  • npm test
  • PYTHONPATH=$PWD/python pytest -q

https://chatgpt.com/codex/tasks/task_e_68590e138d2c832eaae953406af547b1

@ultmaster ultmaster merged commit 622195b into main Jun 24, 2025
3 checks passed
@ultmaster ultmaster deleted the gzm40j-codex/add-trace-dump-for-poml-calls branch July 9, 2025 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant