Repository navigation
feat(factory): link executor PRs to their source issue (0.9.2) - #85
Merged
Merged
Conversation
Tasks carrying github_issue_repo/number get "Fixes #N" (same repo) or "Refs owner/name#N" as the PR's first line, so GitHub closes the issue and PR-side checks like the agent-acceptance Action find the contract the maintainer wrote there. aegis_create_cc_task accepts the link. Governed-By: docs/sandbox-executor.md Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Problem: the executor's PRs carry their acceptance report in the body but close no issue. The agent-acceptance Action reads contracts only from issues a PR closes, and never from the PR body. So on a repo that uses the Action, an executor PR shows as "not machine-checked": seen live on stackbilt-admin/fork-canary#5, which passed 6/6 in the executor.
Change:
issueReference(target, issueRepo, issueNumber): an issue in the PR's target repo becomesFixes #N; an issue elsewhere becomesRefs owner/name#N; a missing or malformed link produces nothing.github_issue_repoandgithub_issue_numberwhen it claims a task, and puts the reference as the PR body's first line.aegis_create_cc_taskacceptsgithub_issue_repoandgithub_issue_number, which must be given together. The executor field it already had is unchanged.Verification:
issueReference(same repo, another repo, malformed links) and the create-tool issue-link validation.🤖 Generated with Claude Code