Skip to content

t1160: Mark Dual-CLI Architecture complete — all subtasks merged#2316

Merged
marcusquinn merged 1 commit intomainfrom
chore/t1160-complete
Feb 25, 2026
Merged

t1160: Mark Dual-CLI Architecture complete — all subtasks merged#2316
marcusquinn merged 1 commit intomainfrom
chore/t1160-complete

Conversation

@marcusquinn
Copy link
Owner

Summary

  • Marks parent task t1160 as [x] complete in TODO.md with pr:#2231 completed:2026-02-25
  • Cleans up bloated BLOCKED notes on t1165.1 (34x repeated dedup guard messages → single clean note)

Context

All 21 subtasks of t1160 (Dual-CLI Architecture) have been implemented and merged across PRs #2053-#2184. The parent checkbox was still unchecked. This PR closes the bookkeeping.

Closes #1746

@gemini-code-assist
Copy link

Summary of Changes

Hello @marcusquinn, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request finalizes the bookkeeping for the Dual-CLI Architecture initiative by marking its overarching task as complete. It also refines the documentation by simplifying a previously cluttered entry related to a subtask, ensuring the TODO.md remains clear and up-to-date.

Highlights

  • Task Completion: Marked the parent task t1160: Dual-CLI Architecture as complete in TODO.md, including the PR number and completion date.
  • Documentation Cleanup: Cleaned up redundant and verbose 'BLOCKED' notes for subtask t1165.1 in TODO.md, replacing multiple identical entries with a concise summary.
Changelog
  • TODO.md
    • Updated t1160 entry to reflect completion, adding pr:#2231 and completed:2026-02-25.
    • Replaced extensive repeated 'BLOCKED' notes for t1165.1 with a single, clear note: 'Cancelled by supervisor. Previously blocked by dispatch dedup guard (t1206).'
Activity
  • No specific human activity (comments, reviews) has been recorded for this pull request yet.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026

Warning

Rate limit exceeded

@marcusquinn has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 8 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.

📥 Commits

Reviewing files that changed from the base of the PR and between 1537794 and 052c2ea.

📒 Files selected for processing (1)
  • TODO.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/t1160-complete

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

🔍 Code Quality Report

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

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

�[0;34m[INFO]�[0m Recent monitoring activity:
Wed Feb 25 22:16:33 UTC 2026: Code review monitoring started
Wed Feb 25 22:16:33 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 90

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 90
  • VULNERABILITIES: 0

Generated on: Wed Feb 25 22:16:36 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

@augmentcode
Copy link

augmentcode bot commented Feb 25, 2026

🤖 Augment PR Summary

Summary: Updates TODO.md to mark t1160 (Dual-CLI Architecture) as completed with recorded PR/date metadata.

Changes: Also replaces a highly repetitive BLOCKED/cancellation note under t1165.1 with a single concise note to keep task history readable.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

- [x] t1159 Fix malformed blocked-by dependency on t1134 #bugfix #auto-dispatch #self-improvement ~10m model:haiku category:data-quality — t1134 shows as blocked-by backtick character (`) instead of a valid task ID. This is likely a YAML/markdown parsing artifact. The task 'Add auto-dispatch eligibility assessment to supervisor AI reasoning' should either have its dependency corrected to a real task ID or have the blocked-by removed if the dependency is resolved. Check the TODO.md entry for t1134 and fix the blocked-by field. ref:GH#1745 pr:#1776 completed:2026-02-19

- [ ] t1160 Dual-CLI Architecture: OpenCode primary + Claude Code CLI fallback #plan #auto-dispatch #architecture #orchestration #cli ~20h model:opus category:infrastructure ref:GH#1746 logged:2026-02-18 — Add Claude Code CLI as a first-class fallback dispatch path alongside OpenCode (primary). Enables OAuth subscription billing for Anthropic model workers, built-in cost caps (--max-budget-usd), native fallback (--fallback-model), and future multi-subscription scaling via containerized instances. See PLANS.md for full design. No regressions to existing OpenCode dispatch.
- [x] t1160 Dual-CLI Architecture: OpenCode primary + Claude Code CLI fallback #plan #auto-dispatch #architecture #orchestration #cli ~20h model:opus category:infrastructure ref:GH#1746 logged:2026-02-18 pr:#2231 completed:2026-02-25 — Add Claude Code CLI as a first-class fallback dispatch path alongside OpenCode (primary). Enables OAuth subscription billing for Anthropic model workers, built-in cost caps (--max-budget-usd), native fallback (--fallback-model), and future multi-subscription scaling via containerized instances. See PLANS.md for full design. No regressions to existing OpenCode dispatch.
Copy link

Choose a reason for hiding this comment

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

This task is being checked off in PR #2316, but the metadata records pr:#2231; is that intentional? If pr: is meant to point to the PR that flips the checkbox/updates TODO.md, this might be a traceability mismatch.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@marcusquinn marcusquinn merged commit 9225fac into main Feb 25, 2026
15 checks passed
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the TODO.md file to reflect the completion of the 'Dual-CLI Architecture' parent task (t1160). The changes include marking the task as complete and cleaning up a long, repetitive note for a subtask, improving the file's readability. The changes are accurate and align with the pull request's goal of closing out completed work.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

t1160: Dual-CLI Architecture: OpenCode primary + Claude Code CLI fallback

1 participant