Repository navigation
Load the committed Project Direction; treat uncommitted triad files as drafts #135
Description
Outcome
Agents always load the committed Project Direction. An uncommitted or edited local VISION.md, MISSION.md or OBJECTIVE.md is reported as a draft and never loaded as the current direction.
Problem Statement
bin/project-direction reads the triad from the working tree and only compares committed HEAD with a known upstream. Uncommitted files are never detected. In toolboxmd/t3code, untracked fleet-pane drafts superseded by a merged direction PR (toolboxmd/chromeria#5) were loaded as the direction in every session for days: the hook reported potentially_stale, but context.md says unknown or stale evidence "never suppresses the local triad", so agents planned against the wrong Objective.
Solution
- Rule (
context.md): the committed triad on the intended base is the Project Direction. Uncommitted or modified triad files in a checkout are drafts. A draft is proposed through the Project Direction procedure (user confirmation, then a PR) or discarded; overwriting a stale local copy is safe because Git keeps every committed state. - Loader (
bin/project-direction inspectand the hooks that call it): per file, compare the working tree withHEAD.- Modified or untracked while
HEADhas the file: inject theHEADversion and report the draft (status such asuncommitted_draft, with the paths). - Known upstream ahead and changing the triad: inject the upstream committed version from local refs (no network, no checkout mutation) and keep reporting it.
- No committed triad anywhere (new project): inject the drafts, marked as drafts, so initialization still works.
- Modified or untracked while
Acceptance criteria
-
context.mdstates the committed-direction rule and the draft lifecycle. - The loader injects committed content for modified and untracked files and reports them as drafts.
- The loader injects the upstream committed triad when known upstream is ahead and changes it, using local refs only.
- New-project initialization with only uncommitted drafts still works, marked as drafts.
- Every host integration that consumes the loader (Claude, Codex, OpenCode, Grok hooks) gets the new behaviour.
- Tests cover clean, modified, untracked, upstream-ahead and never-committed cases.
Non-goals
- Network fetches or checkout mutation by loaders.
- Changing how direction is confirmed or edited.
Blockers
None.
Required proof
- Repository test suite and release-check pass; independent review with a
review/independentsuccess status on the exact head. - A live run of the loader in a scratch repo for each case, output recorded on the PR.
Activity
lukemaj commentedon Sep 27, 2026
Handoff: implementation in PR #136, head
824ba434c9999586df284d3fa293f26e96eb8705onfix/135-committed-direction. Not merged (merging releases 12.5.0).bin/project-directioninjects each triad file fromHEAD, or from the local upstream ref when known upstream is ahead and changes it; checkout copies that differ fromHEADare reported indrafts; a file with no committed version is injected withsourcedraft. All hosts (Claude, Codex, OpenCode, Grok) call this one loader.files[].sourceand optionaldrafts.context.mdstates the committed-direction rule and the draft lifecycle; README paragraph aligned.versionctl release-checkOK (v12.5.0), version-check CI green, live scratch-repo runs for all five cases in feat: load the committed Project Direction; treat uncommitted triad files as drafts #136 (comment).review/independentsuccess on the exact head, posted by an independent reviewer; then human merge approval.feat: load the committed Project Direction; treat uncommitted triad f…
lukemaj commentedon Sep 27, 2026
Handoff update: review findings from #136 (comment) fixed. PR #136 head is now
af13cd7d5c9bb11eb25889f5e5e4eb5841a77376(single squashed commit, v12.5.0). Not merged.HEAD...upstream), so a newer local committed file keepssource: headin a diverged checkout.cat-file --batch --follow-symlinks), with no draft reported.source: draft.release-checkOK, version-check CI green, live runs for the new cases in feat: load the committed Project Direction; treat uncommitted triad files as drafts #136 (comment).review/independentsuccess onaf13cd7, then human merge approval.feat: load the committed Project Direction; treat uncommitted triad f…
lukemaj commentedon Sep 27, 2026
Handoff update: PR #136 head is now
1b29b4dcca3319776713732313966b8eb410f9f8(single squashed commit, v12.5.0). Staleness now follows the upstream-sourced files (stale = bool(upstream_files)): local-only direction commits with unrelated upstream commits reportreadywith no warning. New test fails onaf13cd7and passes here. 302 tests OK,release-checkOK, version-check CI green. Not merged; remaining gate:review/independentsuccess on1b29b4d, then human merge approval.