Repository navigation
chore(audit): delete unreferenced docs (root duplicates, removed-backend era, binaries) - #2560
Conversation
Remove copied root docs, removed-backend guides, and unreferenced binaries. Keep the root policy originals and drop links that pointed at the deleted copies. Co-authored-by: KK <kk-agent@users.noreply.github.com>
|
Please add a See governance: #898 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
🔍 PR Validation |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 68a4574409
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| The original `.zip` also contains `conversation-current-thread.json`, `manifest.json`, | ||
| and `manifest.sha256.txt`. The original download's `assets/WWDC26 gRPC with Swift.txt` | ||
| (WWDC transcript) is preserved inside the zip. |
There was a problem hiding this comment.
Preserve the original conversation source archive
Deleting this directory removes source material, not just stale implementation documentation: the ZIP contains the original WWDC transcript, structured conversation, and integrity manifests. I inspected the archive and searched the target commit's tracked tree; no replacement export or transcript remains. Lack of inbound links does not establish that these source records are disposable. docs/AGENTS.md explicitly requires preserving source fixtures and audit evidence unless a verified cleanup reason requires removal. Retain the original archive, or migrate its source files and integrity metadata to a preserved location before deleting this directory.
AGENTS.md reference: docs/AGENTS.md:L9-L9
Useful? React with 👍 / 👎.
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned Files
|
Keep docs/external/chatgpt-exports as the source archive on main. Put the Cloud rate-limiting link back on the architecture path. Co-authored-by: KK <kk-agent@users.noreply.github.com>
Outcome
Unreferenced and stale docs are deleted. Root originals of the policy files stay. No production path changes.
Closes #2328
#2328 covers three kinds of leftovers: archived folders, deleted-backend Python stubs, and docs tied to removed systems. #2557 (archived folder), #2560 (obsolete docs) and #2565 (deleted-backend stubs) together cover all three, and all three carry this closing keyword.
docs/knowledge_prototypesis the only archived folder left on main. This PR covers the obsolete-docs part.Findings
ARCHITECTURE.md,CHANGELOG.md,SKILL.md,EventRelay-Full-System-Breakdown.md,MERGE_POLICY.md,LAUNCH_CHECKLIST.md,SECURITY.md,CONTRIBUTING.md. Root originals kept.ARCHITECTURE.md,CHANGELOG.md, and the breakdown file were byte-identical to root.MERGE_POLICY.md,LAUNCH_CHECKLIST.md,SECURITY.md, andCONTRIBUTING.mddiffered; the root files are the ones tests and.env.exampleread.git grepfound nodocs/ARCHITECTURE.md,docs/CHANGELOG.md,docs/CONTRIBUTING.md,docs/MERGE_POLICY.md, ordocs/SKILL.mdrefs. Inbound links todocs/LAUNCH_CHECKLIST.mdanddocs/SECURITY.mdwere removed (link line only).SKILL.md(511 lines).UVAIEventBusexisted only there.tests/integration/test_pipelines.pyis absent. The mention indocs/AGENT_CAPABILITIES_CHECKLIST.mdis updated, and the inventory bullet in.github/agent/tasks/cleanup-plan.mdis removed.EventRelay-Full-System-Breakdown.md. Named paths are absent (coordinator.py,universalcoordinator.py,executoraction.py,uvaiintelligence.py,youtubeextension,fabric.py,core/UnifiedCoordinator.py,mcp-servers/python-suite,mcp-servers/shared-state,processors/videoprocessorfactory.py). No remaining filename refs.API_REFERENCE.md,ONBOARDING.md,MASTER_IMPLEMENTATION_GUIDE.md,QUICK_REFERENCE.md,docs/development/(INDEX,ARCHITECTURE_DIAGRAMS,BACKEND_FRONTEND_INTEGRATION,ARCHITECTURAL_REFACTORING_ROADMAP,FASTVLM_SETUP),IMPLEMENTATION_SUMMARY.md,ADVANCED_VIDEO_FEATURES.md,GEMINI_VISION_INTEGRATION.md,TECH_STACK.md,architectural_context.md,DESIGN_BRIEF.md,MONITORING_QUICK_START.md,SENTRY_SETUP.md,MCP_CONSOLIDATION_GUIDE.md,MCP_CONSOLIDATION_SUMMARY.md,YTDLP_OPERATIONS.md. Confirmed missing:src/youtube_extension/backend/api/v1,services/mcp,backend/services/youtube/adapters/robust.py.docs/platform.md(1658 lines, buildpacks platform spec).git grep buildpackoutside that file has no matches.docs/guides/(whole directory, includingrequirements*.txtwith no consumers and the extension-less exports).docs/Code_to_Cameras__Video_AI.mp4(47 MB),docs/MARKETING.pdf,docs/external/chatgpt-exports/,docs/prompts/*.rtf(6 files). No references outside the deleted tree.Link lines removed
docs/SPRINT_DEMO_AGENDA.md,docs/SPRINT_QA_CHECKLIST.md,.github/ISSUE_TEMPLATE/README.md,.github/ISSUE_TEMPLATE/phase-1-security-environment.md,docs/deployment/README.md,.github/agent/plans/implementation_plan.md,docs/CLEANUP_SUMMARY.md,scripts/cleanup_repo.sh.Left in place:
docs/AUDIT_SESSION_CLOSEOUT_REPORT.mdstill namesYTDLP_OPERATIONS.md. Not edited:docs/AUDIT_*is out of scope.scripts/cleanup_bloat.shreason text "Duplicate of API_REFERENCE.md" targets a different file (docs/API_QUICK_REFERENCE.md).scripts/utilities/cleanup_repository.pystill lists the six.rtfnames as root files to move intodocs/prompts/if they reappear. Not a doc link. No test references it..env.examplestill points at rootLAUNCH_CHECKLIST.md, which remains.docs/analysis/,docs/reports/,docs/triage/,docs/pr-remediation/,docs/runbook/,docs/maintenance/,docs/migration/,docs/refactor/,docs/superpowers/,docs/design/enterprise-demo/, anddocs/knowledge_prototypes/were not touched.Verification
Head
68a457440. Docs-only diff. After the commit,git grepfor each deleted path finds no doc links. Remaining hits are the three non-link leftovers above, plus external URLs that contain the substringdocs/guides.CI on this head (run 41670469089):
guards,lint-python,lint-frontend,build,test (Python 3.11),test (Python 3.12), andMCP catalog offline regressionspassed.npm ci --legacy-peer-depsran inside those jobs.test-frontendfailed 2 of 1563 tests inapps/web/src/lib/__tests__/home-sell-surface.test.ts(page copy isStudio prepares proposed work and build rails for you to review.; the test looks forprepares proposed work and build rails for you to review in Studio). The same job already fails onmainatcd7e1998f(run 41659258955). This diff does not touchapps/web.Repository Reconciliation
reportwas cancelled: "Canceling since a higher priority waiting request for repository-reconciliation exists."Scope
Auto-merge is not enabled.