refactor(sdd): align prompt instructions and implement legacy migration#22871
refactor(sdd): align prompt instructions and implement legacy migration#22871moisgobg merged 5 commits intofeat/builtin-conductorfrom
Conversation
|
Hi @moisgobg, thank you so much for your contribution to Gemini CLI! We really appreciate the time and effort you've put into this. We're making some updates to our contribution process to improve how we track and review changes. Please take a moment to review our recent discussion post: Improving Our Contribution Process & Introducing New Guidelines. Key Update: Starting January 26, 2026, the Gemini CLI project will require all pull requests to be associated with an existing issue. Any pull requests not linked to an issue by that date will be automatically closed. Thank you for your understanding and for being a part of our community! |
Summary of ChangesHello, 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 refactors the Spec-Driven Development (SDD) framework to ensure consistent terminology, aligning all internal references with the 'Spec' concept. It removes outdated 'Conductor' and 'Track' terms and introduces automated migration paths for users with existing legacy setups, streamlining the transition to the updated naming conventions and directory organization. Highlights
Using Gemini Code AssistThe 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
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 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. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request does a great job of migrating the Spec-Driven Development (SDD) feature from the legacy "Conductor" and "Track" terminology to the new "SDD" and "Spec" concepts. The introduction of a migration path for legacy conductor/ directories is a thoughtful addition.
However, I've identified a few areas for improvement:
- There are a couple of leftover references to the old "track" terminology in prompt placeholders and path descriptions that need updating for consistency.
- A new
mvcommand introduced for the legacy migration is not whitelisted in the SDD security policy, which will cause the migration to fail. This is a critical issue that needs to be addressed. - One instance of the verb 'specs' is used awkwardly and could be rephrased for better clarity.
Note: Security Review has been skipped due to the limited scope of the PR.
b46e5d7 to
25e26d7
Compare
c0bf255 to
15bb451
Compare
15bb451 to
c879453
Compare
| Plan Mode provides building blocks for structured research and design. Gemini | ||
| CLI includes two built-in planning workflows to suit different project needs. | ||
|
|
||
| ### Standard |
There was a problem hiding this comment.
im hesitant to change unrelated fields in the docs
…with explicit index.md resolution Closes #23017
3d95291 to
f13b179
Compare
Summary
This PR finalizes the migration for Spec-Driven Development (SDD) to a built-in extension. It ensures that the internal prompt instructions, directories, and variables correctly reflect the new "Spec" concept, avoids legacy "Conductor" terminology, and replaces lost abstract protocols with explicit file resolution steps.
Details
tracks: Replaced all instances oftrackandtrackswithspecandspecsacross.tomlcommand prompts and.mdtemplates.conductor: Scrubbed lingering internal references toconductorin prompts and replaced them withsdd.setup.tomlthat detects if a user has a legacyconductor/directory. If found, SDD will prompt the user to migrate the artifacts to the new.gemini/specs/directory automatically. Furthermore, it silently renames any legacytracks/directories ortracks.mdfiles tospecs/andspecs.mdrespectively.GEMINI.mdfile) across all commands (create,implement,revert,review,status). Replaced it with explicit instructions for the agent to resolve project artifacts by reading the.gemini/specs/index.mdfile.Related Issues
tracks#22695conductorwithin prompts #22850/spec setupto account for existing/conductordirectory #22822Universal File Resolution Protocolwith explicit index.md resolution #23017How to Validate
fix/sdd-terminology-cleanup./spec setupand confirm that all prompts, directories, and files (e.g.specs.md,specs/) use the new "spec" terminology.conductor/directory in the root of a project and run/spec setupto trigger and validate the legacy migration prompt..gemini/specs/tracks.mdfile and run/spec setupto ensure it is silently renamed tospecs.md./spec create,/spec status) and observe the agent's behavior. Verify that it correctly reads.gemini/specs/index.mdto resolve file paths without hallucinating or referencing the old "Universal File Resolution Protocol".Pre-Merge Checklist