Skip to content

[codex] Extract PF archive import execution#26

Merged
trek-e merged 1 commit into
mainfrom
codex/pf-archive-import-execution
Jun 16, 2026
Merged

[codex] Extract PF archive import execution#26
trek-e merged 1 commit into
mainfrom
codex/pf-archive-import-execution

Conversation

@trek-e

@trek-e trek-e commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • move PF archive import write/report behavior into a dedicated execution module
  • keep the existing run_import compatibility interface as plan-build plus execution delegation
  • add execution-level tests for existing-NPC skip behavior and dialogue/report writes
  • document the PF Archive Import Execution domain term in CONTEXT.md

Validation

  • uv run ruff check CONTEXT.md modules/pathfinder/app/pf_archive_import.py modules/pathfinder/app/pf_archive_import_execution.py modules/pathfinder/tests/test_pf_archive_import_execution.py modules/pathfinder/tests/test_pf_archive_import_alias.py modules/pathfinder/tests/test_pf_archive_import_integration.py
  • cd modules/pathfinder && uv run pytest tests/test_pf_archive_import_execution.py tests/test_pf_archive_import_plan.py tests/test_pf_archive_import_alias.py tests/test_pf_archive_import_integration.py tests/test_pf_archive_router.py tests/test_pf_archive_router_synthetic.py
  • cd modules/pathfinder && uv run pytest

Notes

  • no uv.lock files changed in this branch

@trek-e trek-e merged commit dd75af4 into main Jun 16, 2026
2 checks passed
@trek-e trek-e deleted the codex/pf-archive-import-execution branch June 16, 2026 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant