ace-assign 0.37.0

data/.ace-defaults/assign/catalog/steps/plan-task.step.yml

@@ -0,0 +1,17 @@
+name: plan-task
+skill: as-task-plan
+description: Analyze task requirements and create an implementation plan
+
+produces: [implementation-plan]
+consumes: [project-base-context, task-spec]
+
+context:
+  default: fork
+  reason: "Planning benefits from focused exploration"
+
+when_to_skip:
+  - "Task is trivial and doesn't need a plan"
+  - "Implementation plan already exists"
+
+effort: light
+tags: [planning, analysis]

data/.ace-defaults/assign/catalog/steps/pre-commit-review.step.yml

@@ -0,0 +1,34 @@
+name: pre-commit-review
+skill: null
+description: Run /review agent command (conversation slash command, not bash) when supported; fall back to ace-lint
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Review should inspect current implementation diff before verification/release"
+
+produces: [pre-commit-review-findings]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Native review execution depends on the current agent/client context"
+
+when_to_skip:
+  - "subtree.pre_commit_review is false"
+  - "No changed files are present in the subtree diff"
+  - "Detected client is not in subtree.native_review_clients"
+
+effort: light
+tags: [review, quality, pre-commit, subtree]
+
+steps:
+  - name: resolve-review-path
+    description: "Resolve review mode from subtree config and detect active client/provider"
+  - name: run-native-review
+    description: "Review uncommitted changes and find issues using the /review agent slash command (NOT a bash command)"
+  - name: summarize-findings
+    description: "Present findings summary and preserve raw output when structure is incomplete"
+  - name: enforce-blocking
+    description: "If pre_commit_review_block is true and critical findings are confidently detected, block downstream release"
+    conditional: "subtree.pre_commit_review_block is true"

data/.ace-defaults/assign/catalog/steps/push-to-remote.step.yml

@@ -0,0 +1,28 @@
+name: push-to-remote
+skill: null
+description: Push all changes to the remote repository
+intent:
+  phrases:
+    - "push to remote"
+    - "push branch"
+    - "publish branch"
+    - "push changes"
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must have commits to push"
+
+produces: [remote-sync]
+consumes: [commits]
+
+context:
+  default: null
+  reason: "Push needs access to git state in main context"
+
+when_to_skip:
+  - "Remote is already up to date"
+  - "Working in local-only mode"
+
+effort: trivial
+tags: [git, publishing, remote]

data/.ace-defaults/assign/catalog/steps/rebase-with-main.step.yml

@@ -0,0 +1,28 @@
+name: rebase-with-main
+skill: as-git-rebase
+description: Rebase current branch onto origin/main while preserving changelog intent
+intent:
+  phrases:
+    - "rebase with main"
+    - "rebase with origin main"
+    - "rebase onto main"
+    - "sync with main"
+
+prerequisites:
+  - name: work-on-task
+    strength: recommended
+    reason: "Rebase is typically done after local work is ready"
+
+produces: [rebased-history]
+consumes: [commits, remote-main]
+
+context:
+  default: null
+  reason: "Rebase rewrites local git history and must run in main context"
+
+when_to_skip:
+  - "Branch already contains latest origin/main"
+  - "Repository policy forbids rebasing this branch"
+
+effort: light
+tags: [git, history, rebase]

data/.ace-defaults/assign/catalog/steps/reflect-and-refactor.step.yml

@@ -0,0 +1,57 @@
+name: reflect-and-refactor
+skill: null  # No dedicated skill — the driving agent follows the steps directly, using ace-review and ace-git-commit as tools
+description: Analyze implementation against ATOM principles and execute targeted refactoring before PR creation
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must have implementation to reflect on"
+  - name: verify-test-suite
+    strength: required
+    reason: "Tests must pass before reflection — refactoring a broken build wastes effort"
+
+produces: [refactoring-commits, findings-report]
+consumes: [code-changes, test-results]
+
+context:
+  default: null  # Inherits from assignment-level context (typically inline, not forked)
+  reason: "Reflection needs access to full diff and project context"
+
+when_to_skip:
+  - "Assignment was trivial (e.g., docs-only, config change)"
+  - "No substantive code changes to analyze"
+  - "Explicitly skipped in assignment configuration"
+
+effort: light
+tags: [quality, architecture, reflection]
+
+steps:
+  - name: demo-validate
+    optional: true
+    description: "Run functional demo proving the feature works end-to-end"
+    skip_when: "verify-test-suite already covers functional validation (E2E tests exist and passed)"
+    on_failure: "Trigger re-plan mechanism instead of proceeding to analysis"
+
+  - name: analyze
+    description: "Review diff using architecture-reflection ace-review preset"
+    tool: "ace-review --preset architecture-reflection"
+
+  - name: decide
+    description: "Produce findings report categorizing items as refactor/accept/skip"
+
+  - name: refactor
+    description: "Execute bounded refactoring based on findings (single iteration, capped scope)"
+    constraints:
+      max_iterations: 1
+      scope: "Only items categorized as 'refactor' in findings report"
+
+  - name: commit
+    description: "Commit refactoring changes separately from implementation commits"
+    tool: "ace-git-commit"
+
+replan:
+  trigger: "Demo validation failure OR architecture analysis finds structural misalignment"
+  distinguished_from: "fix-tests handles test failures; re-plan handles architectural rework"
+  max_reruns: 1
+  max_subtasks: 2
+  after_replan: "Follow-up sub-tasks execute, then reflect-and-refactor runs once more (no further re-plan)"

data/.ace-defaults/assign/catalog/steps/release-minor.step.yml

@@ -0,0 +1,23 @@
+name: release-minor
+description: Release modified packages and update CHANGELOGs following semver
+
+prerequisites:
+  - name: verify-test
+    strength: recommended
+    reason: "Verify changes are stable before releasing subtree output"
+
+produces: [version-bump, changelog-entries, commits]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Version bump needs access to git state and changelog/version files"
+
+when_to_skip:
+  - "No version-worthy changes since last release"
+  - "Release is handled externally"
+
+workflow: wfi://release/publish
+
+effort: light
+tags: [git, versioning, release]

data/.ace-defaults/assign/catalog/steps/release.step.yml

@@ -0,0 +1,23 @@
+name: release
+description: Release modified packages and update CHANGELOGs following semver
+
+prerequisites:
+  - name: work-on-task
+    strength: recommended
+    reason: "Should have code changes before releasing"
+
+produces: [version-bump, changelog-entries, commits]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Version bump needs access to git state and version.rb in main context"
+
+when_to_skip:
+  - "No version-worthy changes since last release"
+  - "Release is handled externally"
+
+workflow: wfi://release/publish
+
+effort: light
+tags: [git, versioning, release]

data/.ace-defaults/assign/catalog/steps/reorganize-commits.step.yml

@@ -0,0 +1,28 @@
+name: reorganize-commits
+skill: as-git-reorganize-commits
+description: Reorganize commits into logical groups for clean history
+intent:
+  phrases:
+    - "reorganize commits"
+    - "clean up commits"
+    - "rewrite commit history"
+    - "tidy commit history"
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must have commits to reorganize"
+
+produces: [clean-history]
+consumes: [commits]
+
+context:
+  default: null
+  reason: "Reorganizing rewrites git history in main context"
+
+when_to_skip:
+  - "Only one commit on branch"
+  - "History is already clean and logical"
+
+effort: light
+tags: [git, history, cleanup]

data/.ace-defaults/assign/catalog/steps/research.step.yml

@@ -0,0 +1,19 @@
+name: research
+skill: as-search-research
+description: Research a topic, codebase pattern, or external documentation
+
+prerequisites: []
+
+produces: [research-findings]
+consumes: []
+
+context:
+  default: fork
+  reason: "Research benefits from focused exploration without side effects"
+
+when_to_skip:
+  - "Topic is already well understood"
+  - "Research was completed in a previous assignment"
+
+effort: medium
+tags: [analysis, exploration]

data/.ace-defaults/assign/catalog/steps/review-pr.step.yml

@@ -0,0 +1,22 @@
+name: review-pr
+skill: as-review-pr
+description: Review code changes for correctness, style, and best practices
+
+prerequisites:
+  - name: create-pr
+    strength: required
+    reason: "Must have a PR to review"
+
+produces: [review-feedback]
+consumes: [pull-request]
+
+context:
+  default: fork
+  reason: "Review benefits from isolated analysis without implementation bias"
+
+when_to_skip:
+  - "No code changes since last review"
+  - "Changes are trivial (typo fix, config update)"
+
+effort: medium
+tags: [review, quality]

data/.ace-defaults/assign/catalog/steps/security-audit.step.yml

@@ -0,0 +1,22 @@
+name: security-audit
+skill: null
+description: Review changes for security vulnerabilities and sensitive data exposure
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must have code changes to audit"
+
+produces: [security-report]
+consumes: [code-changes]
+
+context:
+  default: fork
+  reason: "Security review benefits from isolated, focused analysis"
+
+when_to_skip:
+  - "Changes don't touch security-sensitive code"
+  - "Changes are purely cosmetic (docs, comments)"
+
+effort: medium
+tags: [security, review, quality]

data/.ace-defaults/assign/catalog/steps/split-subtree-root.step.yml

@@ -0,0 +1,25 @@
+name: split-subtree-root
+description: Orchestrate subtree delegation and execution for split parent steps
+
+instructions:
+  common:
+    - "Subtree root orchestrator step."
+    - "This step is orchestration-only."
+    - "Do not execute the parent workflow directly in this parent step."
+    - "Child steps: {{sub_steps}}."
+    - "If subtree execution fails: use adaptive minimal-safe replay (do not replay whole subtree by default)."
+    - "Inject only required recovery steps before retrying failed or affected steps."
+  fork:
+    - "Delegate this subtree into forked context:"
+    - "- ace-assign fork-run --assignment <assignment-id>@{{parent_number}}"
+    - "Inside the forked agent, continue execution within this subtree scope only."
+    - "Note: fork-run may take 10-30 minutes. If your environment has bash timeout limits, run in background and poll scoped status with stepped backoff (5m -> 3m -> 1m -> every 60s)."
+    - "After fork-run completes: read all subtree report files before continuing. The driver is the guard — review reports to verify quality before advancing."
+    - "For failed subtrees: recovery typically includes context refresh (onboard + report review) and verify-test-suite before retry."
+  inline:
+    - "Execute only child steps under this node."
+
+goal_header: "Goal to satisfy through child steps:"
+
+effort: low
+tags: [orchestration, fork, split-step]

data/.ace-defaults/assign/catalog/steps/squash-changelog.step.yml

@@ -0,0 +1,28 @@
+name: squash-changelog
+skill: as-docs-squash-changelog
+description: Squash multiple changelog entries into one before merge
+intent:
+  phrases:
+    - "squash changelog"
+    - "squash changelog entries"
+    - "consolidate changelog"
+    - "merge changelog entries"
+
+prerequisites:
+  - name: release
+    strength: recommended
+    reason: "Most useful after release notes have been generated"
+
+produces: [changelog-entries]
+consumes: [changelog-entries]
+
+context:
+  default: null
+  reason: "Changelog edits and git staging happen in main context"
+
+when_to_skip:
+  - "There is only one relevant changelog entry"
+  - "Changelog has already been consolidated for this branch"
+
+effort: trivial
+tags: [docs, changelog, release]

data/.ace-defaults/assign/catalog/steps/task-load.step.yml

@@ -0,0 +1,29 @@
+name: task-load
+skill: null
+description: "Load task behavioral spec: run `ace-bundle task://<taskref>`"
+
+prerequisites:
+  - name: onboard-base
+    strength: recommended
+    reason: "Needs project context for task interpretation"
+
+produces: [task-spec]
+consumes: [project-base-context]
+
+context:
+  default: null
+  reason: "Task loading needs access to the main conversation context"
+
+instructions: |
+  Load task behavioral spec: run `ace-bundle task://{{taskref}}`
+
+  **Dependency context**: If the task spec has a `dependencies` field listing other task refs:
+  1. For each dependency ref, run `ace-bundle task://<dep-ref>` to load the completed sibling's spec
+  2. Read any work-on-task and plan-task reports for the dependency's subtree from `.ace-local/assign/` reports directory
+  3. This context is critical for planning — it reveals what was already implemented and what to build upon
+
+when_to_skip:
+  - "Task spec already loaded in context"
+
+effort: light
+tags: [setup, context-loading, task]

data/.ace-defaults/assign/catalog/steps/update-docs.step.yml

@@ -0,0 +1,38 @@
+name: update-docs
+skill: as-docs-update
+description: Update public-facing documentation when CLI contracts or public APIs change
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must know what changed to update docs accurately"
+  - name: release
+    strength: recommended
+    reason: "Version number should be known before updating docs"
+
+produces: [updated-docs]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Documentation updates run inline (lightweight, no fork needed)"
+
+when_to_skip:
+  - "No public CLI contract changes (internal refactoring, private methods only)"
+  - "Documentation was already updated as part of work-on-task"
+  - "Task is documentation-only (no new code changes to document)"
+
+effort: light
+tags: [documentation, quality]
+
+steps:
+  - name: identify-scope
+    description: "Identify which packages had CLI/API contract changes by reviewing the diff"
+
+  - name: update-usage-docs
+    description: "Run /as-docs-update-usage for each affected package"
+    tool: "ace-docs-update-usage <package>"
+
+  - name: verify-docs
+    description: "Run ace-docs validate on updated files to confirm format is correct"
+    tool: "ace-docs validate <files>"

data/.ace-defaults/assign/catalog/steps/update-pr-desc.step.yml

@@ -0,0 +1,28 @@
+name: update-pr-desc
+skill: as-github-pr-update
+description: Update PR description with implementation summary and final state
+intent:
+  phrases:
+    - "update pr description"
+    - "update pull request description"
+    - "refresh pr description"
+    - "sync pr description"
+
+prerequisites:
+  - name: create-pr
+    strength: required
+    reason: "Must have an existing PR to update"
+
+produces: [pr-description]
+consumes: [pull-request, code-changes]
+
+context:
+  default: null
+  reason: "Needs access to PR state and git history"
+
+when_to_skip:
+  - "PR description is already up to date"
+  - "No changes since PR was created"
+
+effort: trivial
+tags: [git, pr, documentation]

data/.ace-defaults/assign/catalog/steps/verify-e2e.step.yml

@@ -0,0 +1,42 @@
+name: verify-e2e
+skill: as-e2e-review
+render: step_template
+description: Review E2E coverage for modified packages and run targeted scenarios
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Must have implementation to test"
+  - name: verify-test-suite
+    strength: recommended
+    reason: "Unit tests should pass before running E2E"
+
+produces: [e2e-results, coverage-matrix]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "E2E review and execution run in the project environment"
+
+when_to_skip:
+  - "No public CLI API changes (internal-only refactoring)"
+  - "Package has no E2E test scenarios and no coverage gaps identified"
+  - "All modified files are docs-only (*.md, *.yml non-config, task specs, retros)"
+
+effort: medium
+tags: [testing, e2e, verification]
+
+steps:
+  - name: review-coverage
+    description: "Run /as-e2e-review for each heavily modified package to get coverage matrix"
+    tool: "ace-e2e-review <package>"
+    note: "Identify gaps, overlaps, and which TCs need updating before running"
+
+  - name: update-if-needed
+    description: "If coverage matrix shows gaps or stale TCs, update or create E2E tests"
+    conditional: "coverage-matrix shows gaps or outdated TCs"
+
+  - name: run-targeted
+    description: "Run ace-test-e2e-suite for each heavily modified package (not full suite)"
+    tool: "ace-test-e2e-suite <package>"
+    note: "Pass package name(s) of heavily modified packages only"

data/.ace-defaults/assign/catalog/steps/verify-test-suite.step.yml

@@ -0,0 +1,48 @@
+name: verify-test-suite
+skill: as-test-verify-suite
+render: step_template
+description: Run package test suites with profiling to verify correctness and performance
+intent:
+  phrases:
+    - "run tests"
+    - "run test suite"
+    - "verify tests"
+    - "test changes"
+    - "check tests"
+
+prerequisites:
+  - name: work-on-task
+    strength: recommended
+    reason: "Should have code changes to verify"
+
+produces: [test-results]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Test execution needs access to project environment"
+
+when_to_skip:
+  - "No code changes that could affect tests (documentation-only)"
+  - "Tests were already run and profiled in a previous step"
+
+effort: light
+tags: [testing, verification, performance]
+
+steps:
+  - name: run-package-tests
+    description: "Run ace-test --profile 6 for each modified package"
+    tool: "ace-test --profile 6"
+    note: "Profile reveals slow tests; run per-package, not full monorepo"
+
+  - name: check-performance-budgets
+    description: "Verify no test exceeds documented thresholds"
+    note: "Atoms <50ms, molecules <100ms, integration <1s, full package <30s. Tests must not make real network calls or unnecessary filesystem I/O."
+
+  - name: fix-violations
+    description: "Stub I/O and mock external calls for tests exceeding budgets"
+    conditional: "performance violations detected"
+
+  - name: run-suite
+    description: "Run ace-test-suite to verify no cross-package regressions"
+    tool: "ace-test-suite"

data/.ace-defaults/assign/catalog/steps/verify-test.step.yml

@@ -0,0 +1,36 @@
+name: verify-test
+skill: null
+description: Run profile-guided tests for modified packages only; no full-suite run in subtree
+
+prerequisites:
+  - name: work-on-task
+    strength: required
+    reason: "Implementation should be complete before verification"
+
+produces: [test-results]
+consumes: [code-changes]
+
+context:
+  default: null
+  reason: "Test execution needs access to the project working tree and package contexts"
+
+when_to_skip:
+  - "No code changes in test-relevant files"
+  - "Tests were already profiled for the same subtree run"
+
+effort: light
+tags: [testing, verification, performance]
+
+steps:
+  - name: run-package-tests
+    description: "Run ace-test --profile 6 for each modified package in this subtree"
+    tool: "ace-test --profile 6"
+    note: "Execute per-package; do not run full-stack suite here"
+
+  - name: check-performance-budgets
+    description: "Verify no test exceeds documented thresholds"
+    note: "Atoms <50ms, molecules <100ms, integration <1s, full package suite <30s. Tests must not make real network calls or unnecessary filesystem I/O."
+
+  - name: fix-violations
+    description: "Fix slow or environment-coupled tests when violations are detected"
+    conditional: "performance budgets are exceeded"

data/.ace-defaults/assign/catalog/steps/work-on-task.step.yml

@@ -0,0 +1,23 @@
+name: work-on-task
+skill: as-task-work
+description: Implement task changes following project conventions
+intent:
+  phrases:
+    - "work on task"
+    - "implement task"
+    - "task work"
+    - "build feature"
+
+produces: [code-changes, commits]
+consumes: [project-base-context, task-spec]
+
+context:
+  default: fork
+  reason: "Substantial implementation benefits from focused agent"
+
+when_to_skip:
+  - "Task is documentation-only"
+  - "Changes are already implemented"
+
+effort: medium
+tags: [implementation, core-workflow]

data/.ace-defaults/assign/config.yml

@@ -0,0 +1,48 @@
+# ace-assign default configuration
+# Override in ~/.ace/assign/config.yml or .ace/assign/config.yml
+
+# Directory where assignments are stored
+cache_dir: ".ace-local/assign"
+
+# Main step numbering step (gaps for injection)
+step_increment: 10
+
+# Sub-step numbering format
+subtask_format: "%02d"
+
+# Maximum sub-step depth (e.g., 010.01.01 = 3 levels)
+max_subtask_depth: 3
+
+# Additional skill metadata lookup roots (relative to project root unless absolute)
+# Canonical sources come from protocol registrations in .ace-defaults/nav/protocols/skill-sources/*.yml.
+skill_source_paths: []
+
+# Workflow lookup roots (relative to project root unless absolute)
+workflow_source_paths:
+  - "ace-task/handbook/workflow-instructions"
+  - "ace-assign/handbook/workflow-instructions"
+
+# Fork execution defaults
+execution:
+  provider: "claude:opus@yolo"
+  timeout: 1800
+
+# Subtree execution behavior
+subtree:
+  pre_commit_review: true
+  pre_commit_review_provider: "auto" # auto | native | skip
+  pre_commit_review_block: false
+  native_review_clients:
+    - "claude"
+    - "codex"
+
+# Provider-level defaults for CLI invocation
+providers:
+  cli:
+    - "claude"
+    - "claudeoai"
+    - "gemini"
+    - "codex"
+    - "codexoss"
+    - "opencode"
+    - "pi"