Repository navigation
Document proven session-tool workflow checks - #762
Conversation
…762's P3) manifests/stack.json and the token-efficiency row now use curl --fail --silent --show-error --output "$GRAFANA_HEALTH_FILE" .../api/health; the row says the retained 2026-09-20 observation was taken with the earlier stdout forms (unchanged). Co-Authored-By: Claude Opus 5.5 <[email protected]>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
…egistry plus the owned rows) Co-Authored-By: Claude Opus 5.5 <[email protected]>
…fore the final hot-file commit Co-Authored-By: Claude Opus 5.5 <[email protected]>
… (hot-file protocol: every hot-file edit in the last commit) Co-Authored-By: Claude Opus 5.5 <[email protected]>
|
Claude session native-agent-stack-5f: landing at head Observed main Required checks at this head: 7 pass . Unresolved review threads: 0. |
|
Claude session native-agent-stack-5f: post-merge observation. Landed as |
Scope
Add source-backed checks for task scratch, downloads, terminal launches, native commands and exports, service lifetime, source identity and effective client settings. Each lesson retains its revision or documented scope and its primary reference.
ecfa112764c664d35377dd66b8cfcb67e5a94d60.lane:foundation.docs/token-session-handbook.md; its integrity entry in the finalmanifests/evidence.jsoncommit.SOTA sources
Every added handbook paragraph carries its primary source, reread before publication:
evidence/artifacts/omniroute-sol-max-20260930/checks/preflight-scratch-port-reproduction.jsonat main@ecfa1127; independently parsed one/one/zero returned PID counts.native-agent-stack@ecfa112764c664d35377dd66b8cfcb67e5a94d60:docs/acceptance-evidence-policy.md:24-40,docs/lanes.md:94,recipes/README.md:69,95; gitleaks@83d9cd6 README.Evidence-class table
Local commands run
Heavy checks ran sequentially at niceness 19 with owned temporary storage. Citation corrections were followed by final validation, doc tests and redacted scanning. Returned skips remain recorded. These checks establish doc/integration consistency, not a new model or provider qualification.
Decision record
Live uv, Claude memory/model/CLI and V8 references were reread on 2026-10-06; native Claude Code version/help identified 2.1.291. Scratch placement follows the existing 2026-09-27 “Assuming a temporary directory is outside every Git repository” and 2026-10-02 “Keeping owned worktrees and unpushed receipts in a directory that is emptied at boot” records in
docs/harness-defaults.md:214,306, plusdocs/decisions/2026-10-05-disk-headroom-enospc.md:27-33. The guide explicitly requires an owned persistent directory outside every checkout and repository and qualifies the test guard's/tmpexample by capacity and placement. The probe/configuration links also name their dated lead phrases.Owner follow-up: the generated command manifest's Grafana row and active instruction copies still carry health forms without quiet file output. Their owner should update the canonical generator input and active copies, or route the check through
ctx_execute, while preserving historical command/output evidence unchanged. This change keeps the handbook's corrected form and records that divergence; those files are outside its owned paths.Operating lessons with source-revision recheck conditions, rather than a component-selection change. A changed implementation requires rereading its cited path. Historical claims lacking their original controls remain outside the public guidance.
Host evidence
Source review remains distinct from execution evidence; no host receipt or platform-status promotion.
GitHub checks
The original review-head OSV scan reported five advisory identities across five unchanged base lockfiles (native scan):
blueprints/convergence-practice/application-delivery/uv.lockblueprints/runtime-workers/openhands/requirements.lockblueprints/runtime-workers/openhands/requirements.lockblueprints/us-equities/engine-trials/spy-one-zero-20260926/lumibot/lockcheck/lumibot.lockblueprints/convergence-practice/application-delivery/pnpm-lock.yamlandevidence/artifacts/macos-application-20260924/variant/pnpm-lock.yamlOrdinary and frozen-macOS scans both exited 1. Dependency repair remains separately owned; green checks on a refreshed base are required after it lands. The prior body omitted three advisories; this table corrects that incomplete summary.
Checklist