Skip to content

Extend reviewed CLI fact corrections with resumable chunks - #94

Merged
RaviTejaKNTS merged 2 commits into
mainfrom
feat/cli-app-fact-corrections
Nov 2, 2026
Merged

RaviTejaKNTS merged 2 commits into
mainfrom
feat/cli-app-fact-corrections

Conversation

@RaviTejaKNTS

Copy link
Copy Markdown
Collaborator

Reviewed CLI fact-check batches can now replace platforms, categories, types, pricing/source models, aliases, and install methods. Values use the site's existing option lists, taxonomy, and database rules. Expected values, per-field HTTPS evidence, summary validation, and partial-only updates remain required. Only website/docs URLs, license, and language accept null.

Batches default to rollback and run in transactions of up to 25 apps, with a report after each chunk. Earlier committed chunks survive a later failure. Replaying the same file skips identical fields without app, provenance, date, or cache writes. Provenance writes are combined per app, and query timeouts allow slower links.

Repository URL changes also relink github_project_id in the same transaction, using the refresh script's parser and normalization. Existing GitHub projects are reused without touching their facts; missing projects are created with checked_at null. Non-GitHub URLs clear the link.

Validation:

  • 14 focused tests passed: allowed values, null/HTTPS guards, 300 apps in 12 chunks, rollback/resume, source-write failure, uncertain commit replay, and GitHub project reuse/creation/unlink/rollback.
  • npm run quality passed: TypeScript and 116 core tests.
  • Development database rollback check confirmed accepted fields, nullable columns, GitHub relinking, public-date/cache triggers, full rollback, and zero-write identical replay. No production data changed.
  • No local production build or browser checks: changes affect only the CLI script, its tests, and documentation.

Requires Node.js 22.18 or newer for shared TypeScript options. See scripts/cli-apps/README.md for the batch format and recovery behavior. Not merged; ready for coordinator review.

@RaviTejaKNTS
RaviTejaKNTS merged commit 77e8b21 into main Nov 2, 2026
4 checks passed

This branch was previously deployed

1 inactive deployment
production — 5391315c Deployed Nov 2, 2026 by RaviTejaKNTS via verify-production-build #89
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