Skip to content

Repo maintenance: schema tightening, fork URLs, issue-manager fix, Dependabot, Python 3.13#3

Merged
sherrmann merged 5 commits into
mainfrom
claude/upstream-pr-review-merge-kobghs
Jul 2, 2026
Merged

Repo maintenance: schema tightening, fork URLs, issue-manager fix, Dependabot, Python 3.13#3
sherrmann merged 5 commits into
mainfrom
claude/upstream-pr-review-merge-kobghs

Conversation

@sherrmann

Copy link
Copy Markdown
Owner

Summary

Five maintenance fixes from a full repo audit, one commit each:

  1. Fix issue-manager.yml waiting delay and update to 0.8.0 — the "waiting" delay was 60400 seconds (~17 hours) while its own close message says "after 7 days"; corrected to 604800. Also bumps tiangolo/issue-manager 0.5.1 → 0.8.0.
  2. Point README and Pages site at this fork — the browse URL (sherrmann.github.io/SpoolmanDB) is what gets copied into Spoolman's config, so it must reference this fork's Pages deployment rather than upstream's. Adds a note that the fork carries reviewed but not-yet-merged upstream PR data.
  3. Add Dependabot config — watches github-actions weekly, so future runtime deprecations (like the recent Node 20 → 24 cycle) arrive as automatic PRs.
  4. Reject unknown keys in schemasadditionalProperties: false at every object level of filaments.schema.json and materials.schema.json, so CI catches typo'd or unvalidated keys instead of silently ignoring them. This immediately surfaced a real pre-existing bug: filaments/ratrig.json used spool_material instead of spool_type (fixed here). Validation is now stricter than upstream's, so upstream PRs pulled over in the future may need small fixes first.
  5. Bump CI Python to 3.13 — 3.11 is in security-only maintenance; the compile script runs clean on 3.13 (verified with warnings-as-errors).

Compiled-ID note

Fixing the RatRig typo changes its four compiled IDs from the _n suffix to _p, since the spool type is now recognized. Any Spoolman spools referencing those exact IDs will need re-linking.

Validation

  • check-jsonschema passes on materials.json and all filaments/* against the tightened schemas
  • scripts/compile_filaments.py compiles cleanly, no duplicate IDs
  • All modified YAML files parse

🤖 Generated with Claude Code

https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ


Generated by Claude Code

claude added 5 commits July 2, 2026 08:31
The waiting delay was 60400s (~17 hours) while its close message claims
7 days; 7 days is 604800s. Also bumps tiangolo/issue-manager 0.5.1 -> 0.8.0.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
The browse URL is what gets copied into Spoolman's config, so it must
reference this fork's Pages deployment, not upstream's. Adds a note that
this fork carries reviewed but not-yet-merged upstream PR data.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Adds additionalProperties: false at every object level of both schemas so
CI catches typo'd or unvalidated keys instead of silently ignoring them.
This immediately surfaced filaments/ratrig.json using spool_material
instead of spool_type; fixing that changes the four RatRig compiled IDs
from the _n suffix to _p since the spool type is now recognized.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
@sherrmann sherrmann merged commit 6d37398 into main Jul 2, 2026
3 checks passed
@sherrmann sherrmann deleted the claude/upstream-pr-review-merge-kobghs branch July 2, 2026 08:36
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.

2 participants