Skip to content

[codex] Modernize release standards for Store truth#130

Merged
bnvoss merged 2 commits into
mainfrom
cr-standards-modernization-20260518
May 18, 2026
Merged

[codex] Modernize release standards for Store truth#130
bnvoss merged 2 commits into
mainfrom
cr-standards-modernization-20260518

Conversation

@bnvoss
Copy link
Copy Markdown
Collaborator

@bnvoss bnvoss commented May 18, 2026

Summary

  • modernizes repo-local release standard language from legacy coordination DB and deleted docs/quality release evidence toward Store-first public truth
  • records Store-first runtime truth as project-local .motus/kernel-store.db, runs, events, kind versions, locks, and Work Receipt projections
  • updates standards provenance for motus-work-ledger#35 with recomputed hashes

Evidence

  • git diff --check PASS
  • bash scripts/gates/run-all-gates.sh PASS: 17/17 public gates
  • Headless independent review: APPROVED, no findings

Tracks motus-os/motus-work-ledger#35.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2dc1c737a8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .ai/RELEASE-STANDARD.md Outdated
@bnvoss
Copy link
Copy Markdown
Collaborator Author

bnvoss commented May 18, 2026

Merge evidence for controlled admin path:

  • Scope: .ai/RELEASE-STANDARD.md and .ai/STANDARDS-PROVENANCE.md only.
  • Local validation: git diff --check PASS; bash scripts/gates/run-all-gates.sh PASS 17/17.
  • GitHub checks: check-internal-refs PASS, hygiene-gates PASS, messaging-sync PASS, package-validation PASS, gates PASS.
  • Independent headless review: APPROVED, no findings.
  • Rationale: branch protection cannot represent the separate headless reviewer as a GitHub approval in this solo-maintainer path; proceeding with explicit evidence.

@bnvoss
Copy link
Copy Markdown
Collaborator Author

bnvoss commented May 18, 2026

Follow-up review-thread evidence:

  • Addressed review comment by replacing git status/text scope-lock example with .ai/releases/RELEASE-X.Y.Z-SCOPE-LOCK.json.
  • Scope items must now be explicit: VERSION, MOTUS_WORK_ITEM_ID, and non-empty MOTUS_RELEASE_SCOPE_ITEMS_JSON are required.
  • Each scope item must be a nonblank string; quick validation rejects empty/blank/null/object entries.
  • Targeted headless re-review fix(ci): unblock internal checks and CLI audit #2: APPROVED, no findings.
  • GitHub review thread is resolved and outdated.
  • Final GitHub checks: all PASS.

@bnvoss bnvoss merged commit cd319be into main May 18, 2026
5 checks passed
@bnvoss bnvoss deleted the cr-standards-modernization-20260518 branch May 18, 2026 15:33
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