Skip to content

refactor: clarify proxy task import module - #203

Merged
codekiln merged 1 commit into
mainfrom
codex/202-proxy-task-import-docs
Oct 8, 2026
Merged

codekiln merged 1 commit into
mainfrom
codex/202-proxy-task-import-docs

Conversation

@codekiln

@codekiln codekiln commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Human review of the imported Gitpa sync task found that the helper filename and description hid its purpose. Rename companions.py to task_imports.py and explain how it follows entity/task declarations, plans executable imports, and checks manifest ownership. Document the destination Mise discovery check and update the caller, task-file mapping, and implementation links.

The Proxy Page entity now has an Import manifest section describing its records and linking the existing illustrated explanation. The existing algorithm guide and its accurate historical code citations remain intact. Both edited definition pages were already recorded in today’s journal.

Validation: all 22 proxy tests pass. A new integration test imports the old task layout, refreshes the renamed helper, runs the imported entrypoint, and repeats the import without further writes. The old helper remains a cleanup candidate under the existing policy. Configuration tests cover default discovery, accepted path variants, an excluding include list, and malformed TOML. New graph links resolve; the checker’s existing My/Page/Here example is unchanged. Commit hooks pass.

Closes #202

Addresses human feedback on Gitpa PR #12 helper naming, configuration docstring, and manifest entity documentation. The separate Gitpa consumer PR refreshes these files through the declared sync process.

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.

Clarify proxy task imports and manifest documentation

1 participant