Repository navigation
feat(workflow): align reviews and moderator intake - #9
Merged
Merged
Conversation
Summary Prepare Commit It 3.0.0 for user-approved worker succession independent of chat retention. Changes - Preserve assignment lineage, checkpoints and legacy worker records through approved takeover. - Define OPS delivery receipts and synchronized purpose and work-state labels. - Name conversations through supported client tools and preserve honest review ownership. - Align package documentation and prepare version 3.0.0 without publishing a release tag. Verification - 27 package and installer tests passed; shell syntax and v3.0.0 release checks passed. - Independent eight-scenario review completed; readiness and review ownership ambiguities corrected. - git diff --check passed.
Summary Separate implementation readiness from requested code review and merge readiness. Changes - Default to implementation-worker requests and code review in another conversation. - Permit self-review only under explicit user approval and preserve scoped resolver exceptions. - Add MERGE_READY alongside REVIEW_READY after current review and head/base/check validation. - Invalidate readiness when evidence changes and keep reviews inside implementation scope. Verification - 27 package tests, shell syntax, release metadata and skill validation passed. - Cross-reference and diff checks passed; historical records and version 3.0.0 are retained.
Summary Prepare commit-it 4.0.0 for readable operational records and scoped reviewer repairs. Changes - Specify canonical Markdown records and audited migration without runtime JSON fallback. - Allow requested reviewers to repair the same PR with contributor disclosure. - Allow user-authorized autonomous semantic version maintenance. Verification - Package tests: 28 passed. - Release metadata check for v4.0.0 and git diff --check passed.
Summary Let a separate reviewer complete the user's requested PR review without an implementation-worker request round trip. Changes Document direct user-origin review records, scoped coordination and autonomous repair. Include bounded rebase and exact expected-SHA lease publication before final readiness. Preserve original authors, task separation, runtime restrictions and historical evidence. Verification 28 package tests, shell syntax, release checks and skill validation passed. Independent read-only workflow scenarios covered repair, rebase, drift and runtime refusal. Refs: sungyongcho/ops#11
sungyongcho-ops
added
OPS
Owner
Delivery recorded
|
sungyongcho-ops
marked this pull request as ready for review
September 8, 2026 01:32
sungyongcho
changed the base branch from
codex/readable-review-records
to
main
September 8, 2026 01:46
sungyongcho-ops
marked this pull request as draft
September 8, 2026 01:46
sungyongcho-ops
added
OCCUPIED
Summary Include the current main after its predecessor squash merge. Changes Resolve only history-induced conflicts; retain the exact tested PR file tree. Verification Main tree matches predecessor 65747b5. Result tree matches PR head 900c41a. Existing passing package and CI evidence remains applicable. git diff --cached --check passed.
Summary Align intake coordination with project moderator contracts without creating another backlog or role-based authority bypass. Changes Route designated moderators to project guidance and the existing coordination reference. Classify reports once, size coherent outcomes and preserve single-worker epics. Keep held drafts in chat and distinguish foreign product PRs from owned evidence delivery. Verification 28 package tests and whitespace checks passed. Existing workflow links and approval boundaries were checked against the target contract. Refs: sungyongcho/ops#11
sungyongcho
changed the title
sungyongcho-ops
added
REVIEW_READY
Owner
Delivery recorded
|
sungyongcho-ops
marked this pull request as ready for review
September 8, 2026 02:15
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
2 participants
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.
Summary
The user can direct a separate reviewer to complete bounded review, repair, verification, commits and guarded rebase publication through MERGE_READY without another implementation-worker request. User-origin records preserve original authorship/task provenance; runtime refusals and stale-head protection remain effective.
The user also requested moderator guidance in this PR. The skill now routes designated intake coordinators to the repository contract, sizes issues by coherent outcomes, preserves single-worker epics, keeps held drafts in chat and distinguishes foreign product PRs from owned evidence/policy delivery.
Verification
Refs sungyongcho/ops#11