Skip to content

chore(audit): delete docs/knowledge_prototypes - #2557

Merged
groupthinking merged 2 commits into
mainfrom
cursor/delete-knowledge-prototypes-57cd
Nov 4, 2026
Merged

groupthinking merged 2 commits into
mainfrom
cursor/delete-knowledge-prototypes-57cd

Conversation

@kk-agent

@kk-agent kk-agent commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Canonical issue

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_prototypes is the only archived folder left on main. This PR covers the archived-folder part.

Outcome

The unused docs/knowledge_prototypes/ tree is removed, and the repository no longer refers to that path.

Scope

Risk

  • Risk level: low
  • Failure mode: a maintenance script that relied on the deleted default database path now requires --path.
  • Rollback: revert this commit.

Verification

Local, on this head:

  • rg --hidden --no-ignore -g '!.git/**' knowledge_prototypes finds no matches.
  • python3 -m pytest on the five affected unit modules: 34 passed (-o addopts='' --timeout=120).
  • Frontend lint passed in CI. The frontend test failure is in home-sell-surface.test.ts and does not touch this deletion.

Production evidence

Not applicable. This removes unused prototype docs, a Dependabot directory that no longer exists, and tests that asserted that directory. No production runtime path changes.

Open in Web Open in Cursor 

Remove the unused prototype tree and the tests, Dependabot entry, and default database path that pointed at it.

Co-authored-by: KK <kk-agent@users.noreply.github.com>
@vercel

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Nov 4, 2026 6:57am UTC

@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@github-actions github-actions Bot added ci/cd documentation Improvements or additions to documentation javascript Pull requests that update javascript code python tests labels Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

🔍 PR Validation

⚠️ Large PR detected (36720 lines changed)

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-03T15:47:54.090551Z fe4872d PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

⚠️ Repository reconciliation: this PR does not reference exactly one canonical issue.

Please add a Closes #<issue> reference in the PR description so it can be tracked against the delivery plan.

See governance: #898

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • docs/knowledge_prototypes/mcp-servers/fetch-mcp/package-lock.json
  • docs/knowledge_prototypes/mcp-servers/fetch-mcp/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-code-assistant/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-communication-hub/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-creative-studio/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-data-analysis/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-knowledge-management/package.json
  • docs/knowledge_prototypes/mcp-servers/legacy_proxies/server-workflow-automation/package.json
  • docs/knowledge_prototypes/mcp-servers/mcp-alexnet/requirements.txt
  • docs/knowledge_prototypes/mcp-servers/mcp_youtube-0.2.0/uv.lock
  • docs/knowledge_prototypes/universal-automation-service/monitoring/package.json
  • docs/knowledge_prototypes/universal-automation-service/requirements.txt

kk-agent commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator Author

Reviewed head fe4872d before merge:

  • Scope: 139 deleted files, all under docs/knowledge_prototypes/; 7 modified files only drop references to that tree (dependabot uv entry, its test case, npm-lock snapshot expectation, TECHNICAL_NOTES test, NumPy license note).
  • --path now required in deduplicate_knowledge.py and automated_orchestration_scheduler.py: correct, since the old default pointed into the deleted tree. No other callers in the repo; tests/unit/test_automated_scheduler.py already passes db_path explicitly.
  • Code search on main: every remaining knowledge_prototypes hit is in a file this PR edits.
  • No changes to BACKEND_URL, api.uvai.io, SDKs, or proxy routes.
  • CI: only red checks are test-frontend (1 file failing, home-sell-surface.test.ts, same as main; 175 files pass) and the closing-keyword check. Codex review: no findings.

This branch was successfully deployed

1 active deployment
Preview — 812ae9b0 Deployed Nov 4, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd documentation Improvements or additions to documentation javascript Pull requests that update javascript code python tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove dead archives, backend stubs, and obsolete docs

3 participants