Fix missing format fields and create implementation-plan format#28
Merged
Alan-Jowett merged 1 commit intomicrosoft:mainfrom Mar 19, 2026
Merged
Conversation
Fixes microsoft#19 - Create formats/implementation-plan.md defining the output structure for implementation and refactoring plans (task breakdown, dependency ordering, risk assessment, verification strategy) - Add format: investigation-report to review-code template frontmatter (was in manifest but missing from template) - Add format: implementation-plan to plan-implementation and plan-refactoring template frontmatter - Add format: implementation-plan to manifest for both planning templates - Register implementation-plan format in manifest formats section Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the missing format frontmatter fields to planning/review templates and introduces the previously-referenced implementation-plan format so templates and manifest entries are consistent.
Changes:
- Added
formattoreview-code,plan-implementation, andplan-refactoringtemplate frontmatter. - Added new
formats/implementation-plan.mddefining the implementation/refactoring plan output structure. - Updated
manifest.yamlto register the new format and attach it to planning templates.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| templates/review-code.md | Adds format: investigation-report to template frontmatter. |
| templates/plan-refactoring.md | Adds format: implementation-plan to template frontmatter. |
| templates/plan-implementation.md | Adds format: implementation-plan to template frontmatter. |
| manifest.yaml | Registers implementation-plan format and links it from planning templates. |
| formats/implementation-plan.md | Introduces the new implementation plan output format specification. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #19 — adds missing
formatfields to three templates and creates theimplementation-planformat that was referenced but never existed.Changes
New:
formats/implementation-plan.mdDefines the output structure for implementation and refactoring plans:
Fixed: missing
formatin template frontmatterreview-codeformat: investigation-report(was in manifest, missing from template)plan-implementationformat: implementation-plan(missing everywhere)plan-refactoringformat: implementation-plan(missing everywhere)Fixed: manifest updates
implementation-planto theformats:sectionformat: implementation-planto both planning template entriesValidation
tests/validate-manifest.pypasses.