Repository navigation
feat(collective): show responsibilities before leaving - #94
Merged
Merged
Conversation
Why: Members need an authenticated, privacy-preserving list of unfinished Work, Project, and Artifact custody before the existing transactional leave guard can succeed. The same projection now drives the read path and the final transaction recheck.\n\n[小太阳·砚砚/GPT-5.6 Sol🐾]\nThread-Context: threadId=thread_mtr820c45n7l2b2s catId=codex-sol
Why: The readiness view must carry the authoritative Work status, exact Project source, and current Artifact revision so the Client can explain blockers and return to their real location without guessing from stale projections.\n\n[小太阳·砚砚/GPT-5.6 Sol🐾]\nThread-Context: threadId=thread_mtr820c45n7l2b2s catId=codex-sol
Why: Ship the reviewed Core leave flow from an exact main source inside the plugin and make its generated-byte provenance mechanically verifiable.
Owner
Author
|
@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. |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Outcome
GET /api/memberships/self-leave/readinessprojection that returns all current steward, Work, Roadmap, and custody blockers in one response.Verification
pnpm --filter @clowder-ai/collective-connector typecheck— passed.pnpm --filter @clowder-ai/collective-connector build— passed.test:source— 4/4 passed; the new provenance assertion first failed against deliberately stale metadata after asset replacement, then passed after updating exact hashes.test:installed— fresh packed dependency closure and real SDK lifecycle passed; packaged assets verified.bdc8d5ff472bdcad55a79e6a6f696963eb8167e7, which contains Core PR #5791 merge78431d34c62d495e919023cf94ef43501b645158as its direct parent.git diff --checkpassed; no root media/design artifacts.Five-axis risk
Ownership / activation boundary
Architecture cell remains
collective-runtime; Architecture Map delta is none. Under F202, Service/business contracts and the embedded distribution belong here, while editable Client source stays in cat-cafe. This PR lands source and packaged bytes only; it does not authorize Plugin Manager activation, Service restart, or formal runtime changes.Accepted scope: Task038 messages
thread_mtr820c45n7l2b2s#0001792487226056-001134-deddf11dandthread_mtr820c45n7l2b2s#0001792488334105-001215-d9a17fbc.