Skip to content

Restrict PR creation to axon-workers TaskSpawner only#445

Merged
gjkim42 merged 1 commit intomainfrom
axon-task-441
Feb 26, 2026
Merged

Restrict PR creation to axon-workers TaskSpawner only#445
gjkim42 merged 1 commit intomainfrom
axon-task-441

Conversation

@axon-agent
Copy link

@axon-agent axon-agent bot commented Feb 26, 2026

Summary

  • Inject AXON_TASKSPAWNER env var into agent containers from the axon.io/taskspawner label, so agents know which TaskSpawner created them
  • Remove PR creation instructions from axon-fake-user and axon-fake-strategist prompt templates, directing them to only create issues
  • axon-triage already had the correct restriction ("Do NOT open PRs")

Why

Non-worker TaskSpawners (axon-fake-user, axon-fake-strategist) could create PRs without associated issues or consistent branch naming, making it difficult for maintainers to give feedback (see #441).

Test plan

  • Unit tests added for AXON_TASKSPAWNER env var injection (present and absent label cases)
  • make test passes
  • make verify passes
  • make build passes

Fixes #441

🤖 Generated with Claude Code


Summary by cubic

Restricts PR creation to the axon-workers TaskSpawner. Sets AXON_TASKSPAWNER from the axon.io/taskspawner label to keep PRs tied to issues and our workflow; fixes #441.

  • New Features

    • Inject AXON_TASKSPAWNER when axon.io/taskspawner label is present.
    • Unit tests for label present/absent cases.
  • Bug Fixes

    • Remove PR steps from axon-fake-user, axon-fake-strategist, and the manual fake-strategist task; non-worker prompts now say “Only create issues.”

Written for commit 26262c5. Summary will update on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Copy link
Collaborator

@gjkim42 gjkim42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/reset-worker

git push -u origin axon-fake-strategist-{{.ID}}
gh pr create --title "<title>" --body "<description>" --label generated-by-axon --label ok-to-test
```
- Do NOT create PRs. Only create issues. The axon-workers TaskSpawner handles PRs
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p0: you don't have to mention axon-workers

Copy link
Collaborator

@gjkim42 gjkim42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
/squash-commits

@gjkim42 gjkim42 enabled auto-merge February 26, 2026 15:01
@gjkim42 gjkim42 added this pull request to the merge queue Feb 26, 2026
Merged via the queue into main with commit 54e3bbe Feb 26, 2026
6 checks passed
@gjkim42 gjkim42 deleted the axon-task-441 branch February 26, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

self-development: Axon tasks other than axon-worker should not open a PR

1 participant