Skip to content

Comments

Clean up documentation debris and stale code comments#13

Open
debugmcpdev wants to merge 1 commit intomainfrom
docstar-cleanup
Open

Clean up documentation debris and stale code comments#13
debugmcpdev wants to merge 1 commit intomainfrom
docstar-cleanup

Conversation

@debugmcpdev
Copy link
Collaborator

Summary

  • Delete 4 scratch/empty files from docs/development/
  • Move 65 debris docs to docs/archive/
  • Fix 17 code debris findings across 10 source files

Details

Docstar dogfood found 748 issues. This PR addresses the mcp-debugger cleanup:

Documentation (69 files):

  • Removed scratch files (commit messages, branch lists, secrets setup)
  • Archived test reports, task summaries, investigation notes, refactoring logs

Code debris (17 fixes):

  • Removed stale scaffold/TODO comments from javascript-debug-adapter
  • Removed commented-out code (attachSimplePort, preLaunchTask)
  • Fixed misleading comments (orphan-check, vendor version, log prefixes)
  • Cleaned up placeholder/low-value comments across 10 files

Deferred (9 items): CI debugging blocks and dead code left for maintainer review.

Test plan

  • Verify no source behavior changes (only comments/docs modified)
  • Confirm archived docs accessible in docs/archive/

🤖 Generated with Claude Code

Docstar dogfood cleanup: remove process artifacts, fix code debris.

Documentation (69 files):
- Delete 4 scratch/empty files from docs/development/
- Move 65 debris docs (test reports, task summaries, investigation
  notes, refactoring logs) to docs/archive/

Code debris (17 fixes across 10 files):
- Remove stale scaffold/TODO comments from javascript-debug-adapter
- Remove "REMOVED: attachSimplePort" and commented-out preLaunchTask
- Fix vendor-codelldb.js version comment (1.11.0 → 1.11.8)
- Remove placeholder/low-value comments (adapter-registry, server,
  session-manager-core, dap-proxy-adapter-manager)
- Fix misleading orphan-check comment to match fixed implementation
- Fix inconsistent log prefix in session-manager-data
- Fix module description in session-manager-operations
- Fix @internal JSDoc on public test helper

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant