Frontmatter
| title | chore(ai): add Acceptance Protocol to ticket-intake skill (#10051) |
| author | tobiu |
| state | Merged |
| createdAt | Apr 17, 2026, 8:18 PM |
| updatedAt | Apr 17, 2026, 8:25 PM |
| closedAt | Apr 17, 2026, 8:25 PM |
| mergedAt | Apr 17, 2026, 8:25 PM |
| branches | dev ← agent/10051-ticket-intake-acceptance-protocol |
| url | https://github.com/neomjs/neo/pull/10053 |
Merged

tobiu
commented on Apr 17, 2026, 8:21 PM
Input from Claude Opus 4.6 (Antigravity):
❋ # PR Review Summary
Status: Approved
Clean, focused workflow enhancement. The key insight — that
@mepasses transparently throughIssueService.assignIssue()intogh issue edit --add-assignee "@me"without any service code changes — is empirically validated. This makes the entire PR a documentation/workflow-only change with zero blast radius on runtime behavior.
📊 Evaluation Metrics
[ARCH_ALIGNMENT]: 95 - Follows the meta-level "fix the system that builds the code" philosophy. Two-layer enforcement (intake creates branch, PR workflow validates) is sound defense-in-depth.[CONTENT_COMPLETENESS]: 92 - PR body is comprehensive with empirical verification noted. The workflow documents are well-structured with clear examples.[EXECUTION_QUALITY]: 95 - Minimal, surgical changes across 3 files. No dead code, no over-engineering. The Section 2 → Section 4 redirect on Negative ROI is a good navigational improvement.[PRODUCTIVITY]: 100 - All acceptance criteria from #10051 met, plus the auto-assign enhancement exceeds original scope.[IMPACT]: 70 - High organizational value for preventingdevpollution, but limited to agent workflow (no framework code impact).[COMPLEXITY]: 20 - Straightforward markdown and YAML edits.[EFFORT_PROFILE]: Quick Win - High ROI for minimal complexity. Directly prevents a recurring failure mode.
🕸️ Context & Graph Linking
- Target Epic / Issue ID: Resolves #10051
- Related Graph Nodes:
ticket-intakeskill,pull-requestskill,manage_issue_assigneesMCP tool,IssueService.assignIssue
🧠 Graph Ingestion Notes
[KB_GAP]: N/A — no framework concept gaps.[TOOLING_GAP]: N/A —@meworked transparently without code changes.[RETROSPECTIVE]: The discovery thatghCLI natively resolves@meeliminated an entire code change track. This is a good example of empirical verification (testing before coding) preventing unnecessary complexity. The two-layer branch enforcement pattern (intake gate + PR gate) establishes a robust defense-in-depth model fordevbranch hygiene.
📋 Required Actions
None. This PR is ready to merge.
LGTM ✅
Summary
Adds a mandatory Acceptance Protocol (new Section 3) to the ticket-intake skill that forces agents to auto-assign the ticket and create a feature branch before writing any code.
Problem
Agents were coding on
devand only branching at PR creation time. If a session crashes,devis left dirty. Additionally, there was no mechanism to signal ticket ownership to the swarm, allowing duplicate pickup.Changes
ticket-intake-workflow.md
manage_issue_assignees(action: 'add', assignees: ['@me'])to claim ownershipgit checkout -b agent/[ticket-id]-[descriptor]before any file modificationsdev/main:replace,write_file,multi_replace_file_content,git commitopenapi.yaml
manage_issue_assigneestool description updated to document@meas a special valueassigneesproperty description mentions@meresolves to authenticated usertobiuto@mepull-request-workflow.md
Verification
manage_issue_assignees(action: 'add', assignees: ['@me'])on issue #10051 — confirmed@meresolved totobiuIssueService.mjscode changes needed —@mepasses through transparently togh issue edit --add-assignee "@me"Resolves #10051