Skip to content

intent: portable-core-contracts - #184

Merged
yihanzhu merged 3 commits into
mainfrom
ystack/intent/portable-core-contracts
Aug 29, 2026
Merged

intent: portable-core-contracts#184
yihanzhu merged 3 commits into
mainfrom
ystack/intent/portable-core-contracts

Conversation

@yihanzhu

Copy link
Copy Markdown
Owner

Tracks #155

Intake acceptance: #155 comment 5464340863

What changed

Boundary

This amendment changes only how the work is split and delivered. The five document kinds, three capability IDs, five permission IDs, v1 meanings, canonical bytes, error classes, claims-not-authority boundary, and no-live-activation outcome do not change.

Only work/portable-core-contracts/intent.md changes. No implementation code, spec, plan, issue state, label, workflow, or PR #183 state changes here.

After G1 merge, the old spec and plan are stale by design. Work remains paused until a new G2 spec-with-risk and high-risk plan are independently reviewed and operator-merged.

Proof

  • Exact diff: one file, +41 / -12.
  • git diff --check origin/main...e96b5b7742b1de5a8265109a75f1346696f54d66: clean.
  • Intent blob: f77fc1fdd8f7af228e7f211740901b265fc545ae.
  • Independent pre-PR intent review: PASS; no blockers.

Human merge only. This PR does not merge or resume PR #183.

@yihanzhu yihanzhu added the round-0 Review-loop counter: initial PR label Aug 29, 2026
@yihanzhu

Copy link
Copy Markdown
Owner Author

Codex reviewer (cross-vendor, read-only)

Reviewed-head: e96b5b7
Reviewed-base: 14988a8
reviewer: operator-default @ high

Posted verbatim by codex-review.sh (codex exec review --json --base refs/codex-review/184-59046/base in an isolated temp worktree, sandbox forced read-only). Comments only — Codex never pushes, approves, or merges.

The documentation-only amendment consistently replaces the single-PR delivery constraint with gated, dependency-ordered child parts while preserving the accepted contract and safety boundaries. No actionable regression is evident in the changed lines.

@yihanzhu yihanzhu added the merge-ready Exact reviewed head+base passed; YOUR merge; void if either moves label Aug 29, 2026
@yihanzhu
yihanzhu merged commit d1e403f into main Aug 29, 2026
2 checks passed
@yihanzhu
yihanzhu deleted the ystack/intent/portable-core-contracts branch August 29, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-ready Exact reviewed head+base passed; YOUR merge; void if either moves round-0 Review-loop counter: initial PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant