Skip to content

Commit 287539b

Browse files
Copilotjohnml1135
andcommitted
Add Phase 5 to plan: 30 folders with post-9.3 code changes needing review
Co-authored-by: johnml1135 <13733556+johnml1135@users.noreply.github.com>
1 parent 7256729 commit 287539b

1 file changed

Lines changed: 56 additions & 0 deletions

File tree

COPILOT_SHORTEN_PLAN.md

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,3 +247,59 @@ Total savings: 445 lines removed (620 → 175 lines)
247247

248248
### Completion Status
249249
**ALL 61 FILES COMPLETE** - All COPILOT.md files have been systematically reviewed, condensed, and validated per the strategy documented above.
250+
251+
---
252+
253+
## Phase 5: Post-9.3 Code Changes Review
254+
255+
Analysis of git commits between current branch and release/9.3 identified 30 folders with code changes that may need COPILOT.md updates to reflect new functionality, architecture changes, or added components.
256+
257+
### Folders Prioritized by Change Volume (Needs Review)
258+
259+
**High Priority** - Substantial changes (>30 files changed):
260+
- [ ] Src/Common/ (141 files changed) - Extensive updates across multiple subfolders
261+
- [ ] Src/LexText/ (100 files changed) - Major changes requiring detailed review
262+
- [ ] Src/xWorks/ (40 files changed) - Significant application-level changes
263+
- [ ] Src/FwCoreDlgs/ (39 files changed) - Core dialog changes
264+
- [ ] Src/Utilities/ (32 files changed) - Multiple utility subfolder changes
265+
266+
**Medium Priority** - Moderate changes (10-30 files):
267+
- [ ] Src/XCore/ (25 files changed) - Framework changes
268+
- [ ] Src/ParatextImport/ (20 files changed) - Import pipeline updates
269+
- [ ] Src/FXT/ (11 files changed) - Transform tool changes
270+
271+
**Lower Priority** - Focused changes (4-9 files):
272+
- [ ] Src/FwParatextLexiconPlugin/ (8 files changed)
273+
- [ ] Src/views/ (7 files changed)
274+
- [ ] Src/CacheLight/ (6 files changed)
275+
- [ ] Src/UnicodeCharEditor/ (5 files changed)
276+
- [ ] Src/ManagedVwWindow/ (5 files changed)
277+
- [ ] Src/Paratext8Plugin/ (4 files changed)
278+
- [ ] Src/ManagedLgIcuCollator/ (4 files changed)
279+
- [ ] Src/InstallValidator/ (4 files changed)
280+
- [ ] Src/Generic/ (4 files changed)
281+
- [ ] Src/FdoUi/ (4 files changed)
282+
283+
**Minimal Priority** - Small focused changes (1-3 files):
284+
- [ ] Src/ManagedVwDrawRootBuffered/ (3 files changed)
285+
- [ ] Src/LCMBrowser/ (3 files changed)
286+
- [ ] Src/GenerateHCConfig/ (3 files changed)
287+
- [ ] Src/ProjectUnpacker/ (2 files changed)
288+
- [ ] Src/MigrateSqlDbs/ (2 files changed)
289+
- [ ] Src/FwResources/ (2 files changed)
290+
- [ ] Src/Kernel/ (1 file changed)
291+
- [ ] Src/DebugProcs/ (1 file changed)
292+
293+
### Review Process for Each Folder
294+
1. Run detection script: `python .github/detect_copilot_needed.py --strict --folders Src/<Folder>`
295+
2. If impacted, plan updates: `python .github/plan_copilot_updates.py --folders Src/<Folder> --out .cache/copilot/<folder>-plan.json`
296+
3. Review the plan and git log to understand code changes
297+
4. Apply automated updates: `python .github/copilot_apply_updates.py --plan .cache/copilot/<folder>-plan.json`
298+
5. Manually update narrative sections (Purpose, Key Components, Architecture) to reflect new code
299+
6. Validate: `python .github/check_copilot_docs.py --paths Src/<Folder>/COPILOT.md --fail`
300+
301+
### Notes
302+
- Focus on High Priority folders first (Common, LexText, xWorks, FwCoreDlgs, Utilities)
303+
- Many folders may already have accurate COPILOT.md despite code changes (e.g., bug fixes, refactoring)
304+
- Use git log to determine if changes are substantive enough to warrant narrative updates
305+
- The condensing phase (Phases 1-4) is complete; this phase focuses on accuracy/currency of content

0 commit comments

Comments
 (0)