Skip to content

t3028: Fix task-decompose-helper.sh invocation contract in docs#3042

Merged
marcusquinn merged 1 commit intomainfrom
bugfix/t3028-pulse-decompose-invocation
Mar 7, 2026
Merged

t3028: Fix task-decompose-helper.sh invocation contract in docs#3042
marcusquinn merged 1 commit intomainfrom
bugfix/t3028-pulse-decompose-invocation

Conversation

@marcusquinn
Copy link
Owner

Summary

  • Fix task-decompose-helper.sh invocations in pulse.md, full-loop.md, and headless-dispatch.md to match the actual script API
  • Remove incorrect flags (--task, --repo-path, --task-id, --quiet, --max-depth) and use positional arguments + correct flags (--depth, --max-subtasks)
  • Fix full-loop.md Step 0.45 which incorrectly used classify to check for existing subtasks — now uses has-subtasks subcommand with correct output parsing
  • Fix headless-dispatch.md which referenced a non-existent lineage subcommand — corrected to format-lineage with proper flags

Root cause

The bash examples were written before the helper script API was finalized. The script accepts task descriptions as positional arguments (not --task flags), and uses --depth/--max-subtasks/--lineage (not --repo-path/--quiet/--max-depth). The mismatch meant the helper would treat --task as the description and ignore the real task text.

Files changed

File What changed
.agents/scripts/commands/pulse.md classify and decompose invocations: positional arg, correct flags
.agents/scripts/commands/full-loop.md Step 0.45: use has-subtasks instead of classify for existing subtask check; fix classify and decompose invocations
.agents/tools/ai-assistants/headless-dispatch.md Fix all 3 example invocations to match actual API

Closes #3028

The bash examples in pulse.md, full-loop.md, and headless-dispatch.md used
incorrect flags (--task, --repo-path, --task-id, --quiet, --max-depth) and
a non-existent 'lineage' subcommand. The actual helper API expects:
- classify: positional description, optional --lineage/--depth
- decompose: positional description, optional --lineage/--max-subtasks
- format-lineage (not 'lineage'): --parent/--children/--current
- has-subtasks: positional task-id

Also fixes full-loop.md Step 0.45 which incorrectly used 'classify' to
check for existing subtasks instead of 'has-subtasks', and parsed the
result as JSON with fields that don't exist in the classify output.

Closes #3028
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added the bug Auto-created from TODO.md tag label Mar 7, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 7, 2026

Warning

Rate limit exceeded

@marcusquinn has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 31 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2870d4ac-ec39-4382-97cb-61e78454c18d

📥 Commits

Reviewing files that changed from the base of the PR and between 731a9c7 and 870dd19.

📒 Files selected for processing (3)
  • .agents/scripts/commands/full-loop.md
  • .agents/scripts/commands/pulse.md
  • .agents/tools/ai-assistants/headless-dispatch.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/t3028-pulse-decompose-invocation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 108 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sat Mar 7 05:20:37 UTC 2026: Code review monitoring started
Sat Mar 7 05:20:38 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 108

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 108
  • VULNERABILITIES: 0

Generated on: Sat Mar 7 05:20:40 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 7, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Auto-created from TODO.md tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quality-debt: .agents/scripts/commands/pulse.md — PR #2997 review feedback (medium)

1 participant