Consolidate docs and record multi-operation query reuse - #167
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The recovery guide omits source-preparation quarantine handling and misstates restoration when no prior MCP entry existed.
Review effort: Balanced
Findings: 1
Open (1)
What changed in this PR
Consolidates planning and historical documentation into a single roadmap, durable design guidance, and focused operational guides.
Changes:
- Replaces historical planning pages with concise roadmap/design records.
- Adds multi-operation reuse measurements and current local-testing recovery guidance.
- Repairs documentation links and preserves relevant operational boundaries.
| File | Description |
|---|---|
README.md |
Links to local recovery guidance. |
benchmarks/README.md |
Redirects measurement guidance to design docs. |
docs/README.md |
Updates the documentation index. |
docs/competitive-analysis.md |
Removes the historical comparison page. |
docs/design.md |
Consolidates durable findings and current gates. |
docs/filtrace-etl-trimming.md |
Clarifies that physical trimming is unscheduled. |
docs/local-testing-redesign.md |
Removes the completed redesign history. |
docs/local-testing.md |
Adds current activation and recovery guidance. |
docs/multi-operation-query-reuse.md |
Records reuse measurements and interface options. |
docs/parallelism-opportunities.md |
Removes the closed Track D plan. |
docs/primary-plan-closeout.md |
Removes the superseded closeout record. |
docs/roadmap.md |
Replaces historical queues with concrete triggers. |
docs/source-build.md |
Points retained evidence to consolidated design guidance. |
docs/stack-traversal-experiment.md |
Removes the completed experiment report. |
docs/traceevent-surface-assessment.md |
Removes the superseded dependency inventory. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
docs/roadmap.mdthe one primary work queue, retaining only concrete triggers, open issues, and a short decision/evidence index.docs/design.md; remove six redundant historical pages and speculative LP/TE queues.info+rankCLI measurements, the no-store control, and multi-operation design choices. Explain that MCP already reuses one server-lifetimeTraceStore, and that extra CLI grammar must improve complete agent outcomes rather than just process timing.This is documentation-only. No CLI command, MCP tool, product behavior, raw trace, or publishing contract changes.
Validation
tools/Test-WindowsDotNet.ps1: 1,930 passed, one expected elevated-host skip.The first local test/eval attempts found old unmarked ETLX caches beside test fixtures and generated test-output copies. Those caches were explicitly cleaned with the local CLI before the gates were rerun; no raw fixtures or product code were changed. The current deterministic task count was also corrected to 31.
Review scope
Lite review requested: obvious factual mistakes, broken references, and incorrect operational/recovery guidance. Do not expand this into new feature design or revive the removed investigation queues.