Skip to content

feat: optional TDD mode for SDD workflow tasks - #592

Merged
getappz merged 4 commits into
masterfrom
task/179-feat-optional-tdd-mode-for-sdd-workflow
Aug 22, 2026
Merged

feat: optional TDD mode for SDD workflow tasks#592
getappz merged 4 commits into
masterfrom
task/179-feat-optional-tdd-mode-for-sdd-workflow

Conversation

@getappz

@getappz getappz commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Adds opt-in TDD mode for the SDD workflow, mirroring the existing review_only pattern: WorkItemData.tdd (item-level, computed once at dispatch via a metadata-only detect_tdd_mode check, no free-text scanning needed since this has no legacy callers), passed as an explicit bool parameter into build_implementer_prompt (appends red-green-refactor instructions when set) and build_task_reviewer_prompt (adds a test-first-evidence check to the review criteria when set). Off by default — no behavior change when metadata.tdd is unset. 20 tests pass across prompt_builder_tests, tdd_mode_tests (new), sdd_data_tests, and task_sourcing_tests.


Opened by claude-code on flared:51bb8de6c33b for item #179 via agentflare.

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your current included review allowance is based on your included PR review attempts over the past 7 days.

Next review available in: 3 minutes

Limit details: You’ve used the included review currently available. Your 63 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8c7cf9db-768e-41b7-9d6e-865f491b7227

📥 Commits

Reviewing files that changed from the base of the PR and between a148f73 and 8edd299.

📒 Files selected for processing (4)
  • src/work_item_pipeline.rs
  • src/work_item_pipeline/prompt_builder_tests.rs
  • src/work_item_pipeline/sdd_data_tests.rs
  • src/work_item_pipeline/tdd_mode_tests.rs

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

shiva added 2 commits August 22, 2026 15:40
…etion

item_done previously read ctx.data.reply_text without any assignment,
so it was always empty. synthesize_reply_text() now falls back from
last_report to the ledger and is called at both completion paths
before item_done reads it.

Agentflare-Agent: claude-code
Agentflare-Branch: task/179-feat-optional-tdd-mode-for-sdd-workflow
Agentflare-Item: 179
Adds WorkItemData.tdd (mirroring the review_only pattern), a
metadata-only detect_tdd_mode check, and an explicit tdd bool
parameter on build_implementer_prompt/build_task_reviewer_prompt.
When set, the implementer prompt requires red-green-refactor
discipline and the task-reviewer prompt checks for test-first
evidence. Off by default, no behavior change when unset.

Agentflare-Agent: claude-code
Agentflare-Branch: task/179-feat-optional-tdd-mode-for-sdd-workflow
Agentflare-Item: 179
@getappz
getappz force-pushed the task/179-feat-optional-tdd-mode-for-sdd-workflow branch from bf80721 to 40e5039 Compare August 22, 2026 10:16
shiva added 2 commits August 22, 2026 15:46
…flict resolution

Agentflare-Agent: claude-code
Agentflare-Branch: task/179-feat-optional-tdd-mode-for-sdd-workflow
Agentflare-Item: 179
…ization test to cover tdd

Agentflare-Agent: claude-code
Agentflare-Branch: task/179-feat-optional-tdd-mode-for-sdd-workflow
Agentflare-Item: 179
@getappz
getappz merged commit 6ff6cee into master Aug 22, 2026
16 checks passed
@getappz
getappz deleted the task/179-feat-optional-tdd-mode-for-sdd-workflow branch August 22, 2026 11:34
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.

1 participant