Skip to content

Clean up completed and won't-fix plans - #163

Merged
jeduden merged 4 commits into
mainfrom
claude/cleanup-completed-plans-vrD5k
Apr 22, 2026
Merged

Clean up completed and won't-fix plans#163
jeduden merged 4 commits into
mainfrom
claude/cleanup-completed-plans-vrD5k

Conversation

@jeduden

@jeduden jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner

Summary

Remove 17 completed plans (status: ✅) and 5 won't-fix plans (status: ⛔) from the active planning directory. These documents are no longer part of the roadmap.

Changes

  • Deleted from plan/: 22 plan documents

    • Completed (✅): 50, 51, 62, 64, 66, 69, 73, 74, 75, 76, 77, 79, 80, 81, 82, 87, 88
    • Won't Fix (⛔): 53, 54, 56, 58, 68
  • Updated PLAN.md: Removed rows for deleted plans, now showing only 11 active plans

  • Fixed broken links: Removed references to deleted plans in docs/guides/metrics-tradeoffs.md and plan/91_mds037-skip-generated-sections.md

Result

The plan directory now contains only active and in-progress plans, reducing cognitive load and keeping the roadmap focused on current work.

https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz

Copilot AI review requested due to automatic review settings April 22, 2026 16:45
@jeduden
jeduden force-pushed the claude/cleanup-completed-plans-vrD5k branch from df43f61 to 6f07166 Compare April 22, 2026 16:45
@codecov

codecov Bot commented Apr 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.03%. Comparing base (6554d91) to head (434edd5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   88.03%   88.03%           
=======================================
  Files         110      110           
  Lines       14106    14106           
=======================================
  Hits        12418    12418           
  Misses       1228     1228           
  Partials      460      460           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to declutter the active plan/ directory by archiving completed plan documents (50–88 range) and updating PLAN.md so the active roadmap table no longer lists those completed items.

Changes:

  • Removed multiple completed plan documents from plan/ (intended to be archived).
  • Updated PLAN.md to remove the archived/completed plans from the active plans table.

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
plan/50_redundancy-duplication-detection.md Removed completed plan from active plan/ directory (intended for archive).
plan/51_section-level-size-limits.md Removed completed plan from active plan/ directory (intended for archive).
plan/62_corpus-acquisition.md Removed completed plan from active plan/ directory (intended for archive).
plan/64_spike-go-native-linear-classifier.md Removed completed plan from active plan/ directory (intended for archive).
plan/66_unified-conciseness-score.md Removed completed plan from active plan/ directory (intended for archive).
plan/69_include-enhancements.md Removed completed plan from active plan/ directory (intended for archive).
plan/73_unify-template-directives.md Removed completed plan from active plan/ directory (intended for archive).
plan/74_directive-guide.md Removed completed plan from active plan/ directory (intended for archive).
plan/75_single-brace-placeholders.md Removed completed plan from active plan/ directory (intended for archive).
plan/76_rename-misleading-params.md Removed completed plan from active plan/ directory (intended for archive).
plan/77_template-composition-and-cycles.md Removed completed plan from active plan/ directory (intended for archive).
plan/79_nested-frontmatter-access.md Removed completed plan from active plan/ directory (intended for archive).
plan/80_terminal-recording-readme.md Removed completed plan from active plan/ directory (intended for archive).
plan/81_oom-file-size-limit.md Removed completed plan from active plan/ directory (intended for archive).
plan/82_yaml-billion-laughs.md Removed completed plan from active plan/ directory (intended for archive).
plan/87_markdown-flavor-github-alerts.md Removed completed plan from active plan/ directory (intended for archive).
plan/88_toc-directive-migration.md Removed completed plan from active plan/ directory (intended for archive).
PLAN.md Removed rows for archived/completed plans from the active catalog table.

Comment thread PLAN.md
Comment thread PLAN.md
@jeduden jeduden changed the title Archive completed plans 50–88 to plan/archive/ Clean up completed and won't-fix plans Apr 22, 2026
@jeduden jeduden added queue Add to a PR to enqueue it queue:active Applied automatically when a PR is in an active batch and removed queue Add to a PR to enqueue it labels Apr 22, 2026
@jeduden

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

🟢 Merge Queue — picked up

This PR is in the queue and will be batched with other queue-labelled PRs.

Next: No action needed — you'll get another comment when CI starts on the batch. View merge queue run.

@jeduden jeduden added queue:failed Applied automatically when CI fails or merge conflict occurs and removed queue Add to a PR to enqueue it queue:active Applied automatically when a PR is in an active batch labels Apr 22, 2026
@jeduden

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

⚠️ Merge Queue — merge conflict

This PR could not be merged into the batch branch without conflicts with main or another queued PR.

Next: Rebase onto or merge main into your branch, resolve conflicts, push, then re-add the queue label.

claude added 2 commits April 22, 2026 18:04
Remove 17 completed plans (status: ✅) and 5 won't fix plans (status: ⛔).
Update PLAN.md table via mdsmith to show only 11 active plans.

https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
- Remove plan 53 reference from metrics-tradeoffs.md (won't fix)
- Update plan 50 reference in plan 91 to avoid broken link
- Fix line length issue in plan 91

https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
@jeduden
jeduden force-pushed the claude/cleanup-completed-plans-vrD5k branch from 4c50abd to f5d15ca Compare April 22, 2026 18:04

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

@jeduden Rebased onto main, conflicts resolved, and pushed. Ready to re-queue for merge.


Generated by Claude Code

@jeduden jeduden added queue Add to a PR to enqueue it queue:active Applied automatically when a PR is in an active batch and removed queue:failed Applied automatically when CI fails or merge conflict occurs queue Add to a PR to enqueue it labels Apr 22, 2026
@jeduden

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

🟢 Merge Queue — picked up

This PR is in the queue and will be batched with other queue-labelled PRs.

Next: No action needed — you'll get another comment when CI starts on the batch. View merge queue run.

Copilot AI review requested due to automatic review settings April 22, 2026 18:05
@jeduden

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

🔵 Merge Queue — CI running

Merged into batch branch merge-queue/batch-163-1776881117. View CI run.

Next: No action needed — you'll be notified when CI completes.

@jeduden jeduden removed the queue:active Applied automatically when a PR is in an active batch label Apr 22, 2026
@jeduden jeduden added the queue:failed Applied automatically when CI fails or merge conflict occurs label Apr 22, 2026
@jeduden

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

Merge Queue — CI failed

The batch CI run failed with this PR in it.

Next: Fix the failure, push updates, then re-add the queue label to retry.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR prunes the roadmap by removing completed (✅) and won’t-fix (⛔) plan documents from plan/, and updates the remaining roadmap/index content so in-repo references don’t point at deleted plans.

Changes:

  • Deleted 22 plan documents from plan/ (17 completed, 5 won’t-fix).
  • Updated PLAN.md to list only the remaining active/in-progress plans.
  • Updated documentation/plan text to remove references to deleted plan files.

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
plan/91_mds037-skip-generated-sections.md Removes the cross-plan link to deleted plan 50 (but still needs summary text aligned).
plan/88_toc-directive-migration.md Deleted completed plan doc.
plan/87_markdown-flavor-github-alerts.md Deleted completed plan doc.
plan/82_yaml-billion-laughs.md Deleted completed plan doc.
plan/81_oom-file-size-limit.md Deleted completed plan doc.
plan/80_terminal-recording-readme.md Deleted completed plan doc.
plan/79_nested-frontmatter-access.md Deleted completed plan doc.
plan/77_template-composition-and-cycles.md Deleted completed plan doc.
plan/76_rename-misleading-params.md Deleted completed plan doc.
plan/75_single-brace-placeholders.md Deleted completed plan doc.
plan/74_directive-guide.md Deleted completed plan doc.
plan/73_unify-template-directives.md Deleted completed plan doc.
plan/69_include-enhancements.md Deleted completed plan doc.
plan/68_reorganize-docs.md Deleted won’t-fix plan doc.
plan/66_unified-conciseness-score.md Deleted completed plan doc.
plan/64_spike-go-native-linear-classifier.md Deleted completed plan doc.
plan/62_corpus-acquisition.md Deleted completed plan doc.
plan/58_classifier-model-selection-and-embedding.md Deleted won’t-fix plan doc.
plan/56_spike-ollama-weasel-detection.md Deleted won’t-fix plan doc.
plan/54_metrics-guide-tradeoffs.md Deleted won’t-fix plan doc.
plan/53_conciseness-scoring.md Deleted won’t-fix plan doc.
plan/51_section-level-size-limits.md Deleted completed plan doc.
plan/50_redundancy-duplication-detection.md Deleted completed plan doc.
docs/guides/metrics-tradeoffs.md Removes the planned-metrics table; currently leaves inconsistent “plan 53” references that should be reconciled.
PLAN.md Updates the generated plans catalog to only include remaining plan docs.

Comment thread docs/guides/metrics-tradeoffs.md
Comment thread plan/91_mds037-skip-generated-sections.md
- Update metrics-tradeoffs.md to remove plan 53 reference
- Update plan 91 summary to remove plan 50 reference

https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz

jeduden commented Apr 22, 2026

Copy link
Copy Markdown
Owner Author

Addressed Copilot review comments:

  • ✅ Updated docs/guides/metrics-tradeoffs.md to remove plan 53 reference from disclaimer
  • ✅ Updated plan/91_mds037-skip-generated-sections.md summary to remove "Extends plan 50" reference

All references to deleted plans have been cleaned up. Ready to re-add queue label for merge.


Generated by Claude Code

@jeduden
jeduden merged commit 143275b into main Apr 22, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

queue:failed Applied automatically when CI fails or merge conflict occurs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants