docs(scripts): fix stale --db gittensory example in export-d1-data.mjs#7061
docs(scripts): fix stale --db gittensory example in export-d1-data.mjs#7061philluiz2323 wants to merge 1 commit into
Conversation
The usage comment at the top of scripts/export-d1-data.mjs still told a maintainer to run --db gittensory, the database name from before the 2026-07-15 repo rename. The real D1 database name (wrangler.jsonc's database_name) is now loopover, so copying the example as-is would target a database that no longer exists. Grepped the rest of the file for other stale gittensory-named examples; this was the only one.
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Caution 🛑 LoopOver review result - fixes requiredReview updated: 2026-07-17 20:23:55 UTC
Review summary Nits — 3 non-blocking
CI checks failing
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
|
LoopOver is closing this pull request on the maintainer's behalf (CI is failing (validate, validate-tests (1))). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed. |
Summary
scripts/export-d1-data.mjs's usage comment told a maintainer to runnode scripts/export-d1-data.mjs --db gittensory --output ./export [...]. The real D1 database name (perwrangler.jsonc's"database_name": "loopover") was renamed fromgittensorytoloopoverduring the 2026-07-15 repo rename. Copy-pasting the example as written would target a database name that no longer exists.--db loopover.gittensory-named examples — this was the only one.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Closes #7012
Validation
git diff --checknpm run actionlint(no workflow changes in this PR)npm run typecheck(no type-level surface in a comment-only change; ran anyway, no errors)npm run test:coverage(not applicable;scripts/**is outside the Codecov patch gate percodecov.yml'signore: scripts/**, and per the issue this is a doc-comment-only change with no behavioral diff)npm run test:workers(not applicable to this change)npm run build:mcp(not applicable to this change)npm run test:mcp-pack(not applicable to this change)npm run ui:openapi:check(not applicable to this change)npm run ui:lint(not applicable to this change)npm run ui:typecheck(not applicable to this change)npm run ui:build(not applicable to this change)npm audit --audit-level=moderate(no dependency changes in this PR)If any required check was skipped, explain why:
scripts/export-d1-data.mjs, explicitly called out in the linked issue as "no behavioral diff, no new test required."npm run typecheckandnpm run docs:drift-checkboth pass locally; the existingtest/unit/check-branding-drift-script.test.ts(which scans the repo for stale branding references) also passes with this change applied.Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots. (N/A -- this PR has no UI/visual surface; it corrects a code comment in a maintenance script.)Notes