Skip to content

[AI:codex] docs(workflow): explain the collaboration loop and durable records (TASK-B6B1278890774EF993D2EBC327862470-108) - #3098

Closed
joemooney wants to merge 3 commits into
task-b6b1278890774ef993d2ebc327862470-107-workfrom
task-b6b1278890774ef993d2ebc327862470-108-work
Closed

joemooney wants to merge 3 commits into
task-b6b1278890774ef993d2ebc327862470-107-workfrom
task-b6b1278890774ef993d2ebc327862470-108-work

Conversation

@joemooney

@joemooney joemooney commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Newcomers can follow a change from conversation and captured intent through implementation, validation, independent review and authorized integration. The guide and original SVG distinguish code history, canonical intent, the derived cache and local evidence, and explain maintenance/handoff value without promising transcript capture.

Validation at author head a4ef675ce22cbb165a25c022897e802933723201: make check-ci-fast completed all 35 gates successfully at this exact head. Guide output was rebuilt from its source and shared template; this change adds no runtime authority or deployment behavior. Author checks provide no independent approval or merge-hold clearance. Advisor review and bottom-up integration remain required.

… records (TASK-B6B1278890774EF993D2EBC327862470-108)
@joemooney
joemooney force-pushed the task-b6b1278890774ef993d2ebc327862470-108-work branch from 2e6e624 to 1e6b239 Compare October 30, 2026 00:36
@joemooney
joemooney force-pushed the task-b6b1278890774ef993d2ebc327862470-107-work branch from c85f20f to 26f9a0e Compare October 30, 2026 00:36
@joemooney

Copy link
Copy Markdown
Owner Author

Review: PR-3098 — round 1 (head 1e6b2390b81)

Criterion Verdict Evidence
ac490b57 workflow page connects conversation → intent → owned work → validation → independent review → integration, no single autonomy mode ✅ PASS docs/product-workflow.md:9-16 (six-stage loop + "not a promise that every project uses one autonomy mode")
ace14582 original accessible diagrams of loop + code/store/cache/local-evidence layers; operating model, not dashboard ✅ PASS docs/showcase/assets/workflow.svg (role="img", <title>/<desc>, two panels, footer "Not a live dashboard or backup guarantee"); embedded at workflow.html
ac0a8a25 maintenance/enhancement/handoff value; structured intent ≠ transcript archive / complete observation ✅ PASS docs/product-workflow.md:31-37
ac5e3e02 storage/lifecycle statements match source + references; source links; responsive nav ✅ PASS Claims cross-checked against docs/lifecycle.md (Planned optional, Done on branch, Completed on default branch, Needs Attention), docs/cli/10-storage.md (orphan aida-store, .aida-store/, rebuildable .aida/cache.db), docs/store-location.md (sibling store), aida why --help (file[:line] resolves trace comments). All 8 relative links resolve (incl. OVERVIEW.md#architecture). build.py reproduces all three guide HTML files byte-identically. Nav updated on every guide page + index; .guide-header nav uses flex-wrap; index mobile nav is hidden but #start links to workflow.html.

Gate: PR CI only runs merge-hold-gate here (stacked base ≠ main, and ci.yml triggers only for PRs into main). Ran make check-ci-fast locally at the head: 35 fast-tier gates passed, exit 0.

Non-blocking follow-ups (filed as draft findings)

  • build.py emits a generic <img alt="Workflow diagram; details follow in the source guide">. The SVG's <title>/<desc> isn't exposed through <img>. The prose duplicates the content, so this doesn't block, but a descriptive alt would be better.
  • docs/showcase/README.md still names only getting-started as a generated page.
  • SVG labels are 14px in a 760-wide viewBox, so they render at roughly 7px on a phone.

Note: the spec is blocked-by TASK-106 (Needs Attention), so this PR must integrate bottom-up after #3096/#3097.

Recommendation: Approved, merge in stack order.

…0774EF993D2EBC327862470-108)

# Conflicts:
#	docs/showcase/index.html
@joemooney

Copy link
Copy Markdown
Owner Author

Final-round author update at a4ef675ce22cbb165a25c022897e802933723201.

Merged the final corrected predecessor and regenerated affected guide/library artifacts. No new slice-specific content edits; final approval/base-only attestation remains the advisor’s decision.

All lower-stack fixes are included by ordinary merges; no rebases or force pushes. These are author checks, with no independent-review or hold-clearance authority.

Validation: make check-ci-fast completed all 35 gates successfully at this exact head

@joemooney

Copy link
Copy Markdown
Owner Author

Included in PR-3103, which merged the approved stack 107-113 to main as one PR (advisor 11:35).

@joemooney joemooney closed this Oct 30, 2026
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