Repository navigation
chore(audit): remove deleted-backend entrypoints, scripts and deploy files - #2567
Conversation
Drop silent imports, CLI commands, console scripts, and retired deploy files that still targeted the backend package removed in #2320. Co-authored-by: KK <kk-agent@users.noreply.github.com>
The pip directory /src/youtube_extension/backend no longer exists. Co-authored-by: KK <kk-agent@users.noreply.github.com>
|
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. |
|
Please add a See governance: #898 |
🔍 PR Validation |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Delete the no-op tracker and the timing that existed only to call it. Co-authored-by: KK <kk-agent@users.noreply.github.com>
Canonical issue
Closes #2325
#2325 ("Remove unwired Python backend and duplicate Video Pack emitter") was filed alongside #2319, whose backend and emitter deletion shipped in #2320. The pieces that still booted, packaged or deployed that deleted backend (entrypoints, console scripts,
alembic.ini, Cloud Run and compose tombstones) are what this PR removes, which completes #2325. Related: #2328.Outcome
Entrypoints, scripts, and deploy files that still boot, package, or target the Python backend deleted in #2320 are gone.
/readyzreports only that the remaining process is accepting traffic. The live UVAI backend at api.uvai.io (BACKEND_URL) runs outside this repo and is not redeployed or reconfigured by this diff.Scope
src/youtube_extension/main.py: removed the four silent imports of deletedbackendmodules,_API_V1_ROUTER_LOADED,ensure_api_cost_ready, and the sqlite bootstrap helper that only that probe called./readyzreturns{"status": "ready", "service": "uvai-backend"}.src/youtube_extension/cli.py: removedapi-cost-worker,migrate, and the doctor probe that importedHealthMonitoringServicefrom the deleted backend.pyproject.toml: removed console scriptsuvai.api,uvai.core,youtube-worker, andyoutube-migrate(the last one pointed at the deletedmigratecallable). Project URLs now point athttps://github.com/groupthinking/EventRelay. Dependency lists are unchanged.alembic.ini(script_locationwas the missingsrc/youtube_extension/backend/migrations).README.md: Video Pack sentence no longer calls the Python backend a canonical writer. Removed the paragraph that described the deleted sqlite readiness bootstrap.BACKEND_URLsentence kept.scripts/nightly_audit_agent.py: removed the deadyoutube_extension.backend.services.*import block and rewrote the module docstring to the log scan the script still performs. Script kept.scripts/skill_builder.py: removed the__main__seed that named deletedapi_cost_monitor.infrastructure/database/index_analysis.py: removed the dead backend import, the no-optrack_database_query_timeshim, and the two timing blocks that existed only to call it. Analysis queries stay.scripts/utilities/test_cicd_pipeline.py,scripts/utilities/fix_syspath_issues.py,scripts/utilities/calibrate_env.py,scripts/testing/verify_real_api_integration.py.deploy-cloud-run.yml:scripts/deployment/deploy-cloud-run.sh,infrastructure/cloudrun/{service.yaml,setup.sh,deploy.sh,cloudbuild.yaml}.infrastructure/docker/Dockerfile.autonomous(simple_real_processor.pydoes not exist).infrastructure/docker/docker-compose.prod.ymlanddocker-compose.full.ymlafter confirming default compose-file-relative paths do not resolve (infrastructure/docker/Dockerfile,infrastructure/docker/infrastructure/database/init.sql, andfactory/celery_app.pyare missing). Neither file names api.uvai.io..github/dependabot.yml: removed the pip directory/src/youtube_extension/backend. Root/and/sdk/pythonpip blocks stay.tests/unit/test_dependabot_automation_workflow.py: dropped the parametrize case for that deleted directory..env.examplekeys, includingAPI_COST_TRACKINGandBACKEND_URL. Skipped: env keys can still configure the live host outside this repo.scripts/dev_backend.sh. Skipped: it execspython -m uvicorn youtube_extension.main:app, which still exists. The brief finding does not hold.infrastructure/cloudrun/domain-mapping.yamlandinfrastructure/cloudrun/setup-secrets.sh. Skipped: they route and name credentials for the live api.uvai.io Cloud Run service.Dockerfile,infrastructure/docker/Dockerfile.production,infrastructure/terraform/,infrastructure/k8s/,scripts/deployment/one-click-deploy.sh,.github/workflows/*,src/agents/*.src/youtube_extension/services/ai/hybrid_processor_service.pystill imports deletedyoutube_extension.backend.services.api_cost_monitor.track_api_callinside_record_api_usage. Observed and left in place; it is not an entrypoint, script, or deploy file in the brief.Findings
Done:
main.pysilent backend imports and_API_V1_ROUTER_LOADED.ensure_api_cost_readyand the/readyzcall into it, plus the sqlite bootstrap helper and the README paragraph that described it.api-cost-worker,migrate, and the doctorHealthMonitoringServiceimport.pyproject.tomlscriptsuvai.api,uvai.core,youtube-worker, and the follow-through removal ofyoutube-migrate. Project URLs rewritten to groupthinking/EventRelay, including Changelog.alembic.ini.scripts/nightly_audit_agent.pydead imports and docstring.scripts/skill_builder.py__main__seed.infrastructure/database/index_analysis.pydead backend import, the no-op query-time shim, and the timing that only fed that call.scripts/utilities/test_cicd_pipeline.py,fix_syspath_issues.py,calibrate_env.py, andscripts/testing/verify_real_api_integration.py.scripts/deployment/deploy-cloud-run.shandinfrastructure/cloudrun/{service.yaml,setup.sh,deploy.sh,cloudbuild.yaml}after confirming they are retired tombstones and do not name api.uvai.io.infrastructure/docker/Dockerfile.autonomous.infrastructure/docker/docker-compose.prod.ymlanddocker-compose.full.ymlafter confirming broken relative paths and no live-host reference./src/youtube_extension/backend, and the unit test case that asserted it.Skipped:
.env.examplekey removals. The live backend at api.uvai.io may still consume them.BACKEND_URLstays.scripts/dev_backend.sh. It still starts the remaining FastAPI app.infrastructure/cloudrun/domain-mapping.yamlandsetup-secrets.sh. They configure routing and secrets for the live host.Risk
uvai.api,uvai.core,youtube-worker,youtube-migrate) or of the deleted compose/cloudrun tombstones fails at invocation. Those targets already failed or exited 1.Verification
Head
e6c6fb64bbc15e2baa985d21d9b814a348d8b1d2.python -m compileall -q src/uv lock --checkwith uv 0.12.1 (Resolved 346 packages). Dependency lists were not edited.PYTHONPATH=src python -m pytest tests/unit/test_index_analysis.py -q --timeout=120 -o addopts=''— 5 passed.PYTHONPATH=src python -m pytest tests/unit/ -q --timeout=120 --cov=src/youtube_extension --cov-fail-under=0 --ignore=tests/unit/test_transcript_action_workflow.py -k "not integration"— 3051 passed on508abce70. One failure was the dependabot case for the deleted directory, removed in508abce70. Eleven other failures werepython: command not foundon this VM (python3only); the same tests passed with/tmp/er-venv/binonPATH. CI runners providepython.python -m pytest tests/security/test_local_mcp_config.py tests/security/test_mcp_catalog_approval.py -q -o addopts=''— 38 passed.508abce70: Python 3.11 and 3.12 unit tests, lint-python, lint-frontend, build, guards, MCP catalog, coverage, dependency review, CodeQL, bandit, safety, trivy, npm audit, gitleaks, and E2E passed.e6c6fb64bonly deletes the index-analysis shim and its two call sites.home-sell-surface.test.ts, which expects the page source to containprepares proposed work and build rails for you to review in Studio.apps/web/src/app/page.tsxsaysStudio prepares proposed work and build rails for you to review.This PR does not changeapps/web. The same job is already the only failure onmainatcd7e1998f(https://github.com/groupthinking/EventRelay/actions/runs/41659258955). Left unfixed.Production evidence
Not applicable. This deletes in-repo leftovers of a backend that already does not live in this tree. It does not deploy, change domains, change secrets, or change
BACKEND_URL. api.uvai.io stays on its existing host outside the repo.Agent handoff
508abce70passed. test-frontend is red onmainfor the same home-copy assertion (fixed by chore(audit): fix stale home-sell-surface test #2568) and was not changed here.