fix: correct drifted inventory counts and contributor docs - #190
Merged
Merged
Conversation
The game-asset category header, the contributing guide's inventory, and the security scope still described an older tree. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Authoring docs still said to add a blender-smoke.yml step and cited skill and rule paths that are not in the tree. Smoke rows live in the catalog, and the manifest check includes showcase. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
The template lives in .github, so a same-directory CONTRIBUTING.md link does not resolve. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Documentation audit against the tree at
v0.78.16. Mechanical drift is corrected. Decisions are filed and not changed.LICENSE,VERSION,CHANGELOG.md, andrelease.ymlwere not edited. No example or showcase script changed.live-run-proven: disk counts, markdown link scan, and HTTP checks (Python, this machine). GitHub About read and patched via
gh api.inspection-only: version-stamp comparison, index diffs, falsifier wording, rule frontmatter versus the CLAUDE scope column, branch-protection API, issues #131 and #146.
1. Findings
examples/tests/smoke/run_example.pyskills/procedural-materials/andrules/avoid-python-loops-on-vertices.mdcblender-smoke.ymlsteptests/smoke/catalog.jsonCONTRIBUTING.mdfrom.github/../CONTRIBUTING.md0.78.16; not editedtests/check_exit_code_readme.pyprojects.blender.org/lab/blender_mcpreturns 403label-sync.ymlignoresexamples/andshowcase/alwaysApply: trueon every rulepages.ymllistsmcp-tools.json, which is not on diskrelease.yml; not filedmainhas no branch protection and no rulesets2. Recount
Disk, remeasured this run: 16 skills, 9 rules, 27 snippets, 3 templates, 59 examples, 26 showcase pieces.
validate-countsgreps README only. Those aggregate substrings already matched, so this PR does not change what that job asserts except that the game-asset header is not part of the needle.3. Version stamps
VERSIONCHANGELOG.mdtop headingCLAUDE.md**Version:**ROADMAP.md**Current:**plugin.jsonversion4. Links
119 markdown files plus 9 rules. 417 internal link occurrences, 2 anchor-only. Broken file links: 1, fixed. Broken anchors: 0.
External markdown occurrences: 167. HTTP checks: 243 (unique externals plus every published
docs/gallery/*/index.htmlpage and the site root). Non-200: 1..github/PULL_REQUEST_TEMPLATE.md:33CONTRIBUTING.md.github/../CONTRIBUTING.mdhttps://projects.blender.org/lab/blender_mcpBadge workflows
validate.yml,blender-smoke.yml, anddrift-check.ymlexist. Homepagehttps://tmhsdigital.github.io/Blender-Developer-Tools/returned 200 and was not changed.5. Manifest consistency
No index entries added or removed.
plugin.jsonvs disktests/smoke/catalog.jsonvse-cut-list-pixelsis a second witness ofexamples/vse-cut-list/examples/gallery.jsonvspreview.webpngon-triangulate,unapplied-scale-gltf,coincident-vert-weld,vse-linear-modifiers,eval-mesh-datablock-name,mesh-automasking-settings,gn-bundle-roundtrip,exit-pre-sidecarshowcase/gallery.jsonvspreview.webpThose eight exclusions are documented on the example READMEs and in ROADMAP. There is no single exclusion index. None was added.
6. Contradictions beyond #146
Fixed in this PR: smoke registration described as a workflow step while the runner reads the catalog; CONTRIBUTING's example paths named files that are not in the tree; AGENTS described validate-manifest as if showcase were not checked.
Still open, and not edited: #146.
docs/new-example-prompt.mdstill requires a framed render, and CLAUDE still tells authors to wire a gallery still, while eight examples are check-only.CLAUDE.mdstill says "their renders ship in the site gallery" for the example set as a whole. That sentence was left alone so this PR would not pick a side of #146.degenerate-bevel-weldfalsifies with--both-safeand does not use the word "falsifier". The other 58 example scripts do. Exit-code tables are already enforced bytests/check_exit_code_readme.py. Showcase READMEs all declare budgets and exit codes. Onlytemplates/ai-asset-pipeline-template/pipeline.pyduplicates snippet helpers; its header names the sources.7. Onboarding read-through
The first paragraph says what this is: skills, rules, snippets, templates, and runnable examples for Blender Python development, aimed at 5.2 LTS with 4.5 fallback and 5.1 as prior stable.
Quick start clones the repo and runs
examples/bmesh-gear/bmesh_gear.pyheadless. Someone withblenderon PATH can do that without leaving the README. It does not say where to get Blender.Showcase is in the headline count, in its own section, and in the layer table, which says it is not examples and points at
showcase/README.md.The falsifier idea is not in the README. It shows up in
CLAUDE.mdquality gates, inCONTRIBUTING.md, and in the PR template ("falsified once").The overview says any MCP-capable client consumes this content. This repository has no MCP server. Cursor and Claude Code do load the files directly. That phrase is loose, not a missing feature, so it was left as written.
8. Issues
examples/andshowcase/alwaysApply: true9. Outside the diff
GitHub description patched from
13 skills, 6 rules, 2 templates, 17 snippets, 45 examplesto16 skills, 9 rules, 3 templates, 27 snippets, 59 examples, 26 showcase pieces. Homepage left athttps://tmhsdigital.github.io/Blender-Developer-Tools/.10. Health
In good shape: version stamps, the four indexes, the exit-code AST gate, Blender version targeting, skill frontmatter, and link integrity after the one internal fix.
Drifting, and worth auditing first next time: the GitHub description (it was stale again), category headers that
validate-countsdoes not grep,SECURITY.md's support table, the gallery noun, rulealwaysApplyversus the documented scope, and whether #131 and #146 are still open. #131 has widened since it was filed.Test plan
validate/validate-manifest/validate-counts/validate-harnessgreenneeds-5.1)