docs(contributing): document every test:ci check, not just three - #3579
Conversation
The contributing skill's generated-artifacts list and CI check table only mentioned ui:openapi/cf-typegen/migrations, but package.json's test:ci script runs eight more drift/build checks (db:schema-drift, selfhost:env-reference, selfhost:validate-observability, build:miner, test:miner-pack, rees:test, docs:drift-check, command-reference) that were undocumented and could surprise a contributor who trusted the skill's checklist as complete.
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-05 17:41:10 UTC
⏸️ Suggested Action - Manual Review
Review summary Nits — 3 non-blocking
Concerns raised — review before merging
Review context
Contributor next steps
Signal definitions
🟩 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 Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Summary
.claude/skills/contributing-to-gittensory/{SKILL.md,reference.md}andAGENTS.md's"regenerate generated artifacts" guidance only named
ui:openapi/cf-typegen/migrations.package.json's actualtest:ciscript runs eight more checks that weren't documented anywherein the skill:
db:schema-drift:check,selfhost:env-reference:check,selfhost:validate-observability,build:miner,test:miner-pack,rees:test,docs:drift-check,command-reference:check.reference.md's check table, addedselfhost:env-referenceandcommand-referencetoSKILL.md's artifact-regeneration table, correctedSKILL.md'stest:cienumeration to match the real script, and pointed
AGENTS.md's "five things" docs(site): align public docs with gittensor aesthetic #4 at the full list.Test plan
.claude/skills/**andAGENTS.md; nosrc/**impact.package.json's actual script definitions.