Skip to content

docs(policies): point docs at docs/policies/ and document trajectory_hint exception (#68) - #73

Merged
tishachawla-jg merged 1 commit into
theagentplane:mainfrom
amasen02:fix/docs-policies-docstring-path-68
Sep 3, 2026
Merged

docs(policies): point docs at docs/policies/ and document trajectory_hint exception (#68)#73
tishachawla-jg merged 1 commit into
theagentplane:mainfrom
amasen02:fix/docs-policies-docstring-path-68

Conversation

@amasen02

@amasen02 amasen02 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #68 — updates src/tokenops/control/policies/__init__.py to point governance documentation to docs/policies/ instead of non-existent tokenops-lld/policies/, and notes the trajectory_hint configuration exception.

Changes

  1. Corrected docstring path reference from tokenops-lld/policies/ to docs/policies/.
  2. Documented why trajectory_hint is not exported alongside the other policy templates (requires a Store and is configured in src/tokenops/control/config.py).

Checklist

  • Corrected documentation link path
  • Documented trajectory_hint design exception
  • Staged as Draft PR

…hint exception (theagentplane#68)

- Update policies docstring to point to docs/policies/ instead of non-existent tokenops-lld/policies/
- Document that trajectory_hint requires a Store and is configured via config.py
@tishachawla-jg

Copy link
Copy Markdown
Collaborator

Verified against current `main` and both changes hold:

  • `tokenops-lld/` does not exist in this repo; `docs/policies/` does, with 11 pages. The docstring pointed at a dead path.
  • `trajectory_hint` is confirmed absent from this module's imports/`all`, and is exactly where you said: special-cased in `src/tokenops/control/config.py` because it needs a `Store`.

Ran the full suite on this branch: ruff clean, ruff format clean, mypy clean on 54 files, 218 passed, 2 skipped.

I approved the blocked CI run on the fork, it's now green on 3.10/3.11/3.12.

This is marked as a draft, thanks for flagging it that way. Ping this thread (or mark it "Ready for review") whenever you want it merged, and it'll go in as-is; nothing here needs changes from my side.

@tishachawla-jg
tishachawla-jg marked this pull request as ready for review September 3, 2026 14:26
@tishachawla-jg
tishachawla-jg merged commit 5ee3b3e into theagentplane:main Sep 3, 2026
3 checks passed
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.

policies/__init__.py points docs at tokenops-lld/policies/, which does not exist

2 participants