feat(client): add offline semantic release admission contract - #5
feat(client): add offline semantic release admission contract#5seonghobae wants to merge 80 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Reserve ADR 0004 for Source Observation and move the Client boundary to ADR 0005. Deduplicate LLVM source regions across integration-test binaries and restore the documented detached-artifact contract. Signed-off-by: Seongho Bae <me@seonghobae.me>
|
Current-head repair |
|
@codex review |
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0087f6c8bd
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0087f6c8bd
ℹ️ 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".
# Conflicts: # ARCHITECTURE.md # CHANGELOG.md # Cargo.lock # Cargo.toml # docs/PRD.md # docs/TRD.md # docs/adr/README.md
Purpose
Advance Issue #3 through the Rust-first, provider-independent Client Consumption bounded context. Downstream products consume governed immutable
semantic_releasecontracts without importing generator internals, provider payloads, persistence tables, or orchestration state.Current exact stack — 2026-09-05
b538470c963e6524ddc0c3f652a46a4fc8265150;fcf36c8a99f015b963c9f812787df127ac2e2f9e;Current contract
The source retains a versioned public semantic-release schema, owned 100% coverage enforcement, structural plus deterministic semantic supersession validation, immutable-content collision rejection in
SemanticReleaseClient::diff, and explicit predecessor/successor lifecycle handling. Client Consumption owns generic release admission, integrity, compatibility, diff, resolution and supersession validation; consuming products retain tenant/purpose authorization, business truth and physical execution.semantic-data-portalremains catalog/governance/consumption,context-graph-contractsinterop,enterprise-architecture-coreEA, andcontextual-orchestratorproduction LLM/provider routing. These deterministic validators make no model call.Merge gate
Keep Draft until Foundation is independently integrated and this exact successor has terminal Product/security/SAST/dependency/review evidence plus zero valid unresolved findings. No self-approval, routine admin bypass, fail-open scanner substitution, force-push, destructive rebase, mutable supplier dependency, or predecessor-evidence transfer.