Skip to content

Add no_bat_spawn behavior pack - #12

Merged
Ven0m0 merged 4 commits into
mainfrom
claude/plan-md-implementation-oufb1h
Aug 16, 2026
Merged

Add no_bat_spawn behavior pack#12
Ven0m0 merged 4 commits into
mainfrom
claude/plan-md-implementation-oufb1h

Conversation

@Ven0m0

@Ven0m0 Ven0m0 commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Data-driven spawn_rules override with an empty conditions array,
replacing vanilla's minecraft:bat spawn rules so bats never spawn
naturally. No script module needed, unlike silk_touch_drop. Spawn
eggs, /summon, and monster spawners are untouched by design.

Update README.md and AGENTS.md for the two-pack repo.## Summary

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Enhancement (improvement to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactor (no functional changes)
  • Chore (maintenance, dependencies, CI)

Related Issues

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • New and existing tests pass locally
  • Any dependent changes have been merged and published

Screenshots / Recordings

Pre-Merge Checklist

  • Self-reviewed the diff
  • Commit messages follow project conventions
  • Added/updated tests covering the change
  • All CI checks pass
  • Updated docs / README / CHANGELOG (if applicable)
  • No unrelated changes bundled in
  • Reviewed for security implications
  • Breaking changes documented and communicated

Reviewer Notes

claude added 4 commits August 16, 2026 06:36
Data-driven spawn_rules override with an empty conditions array,
replacing vanilla's minecraft:bat spawn rules so bats never spawn
naturally. No script module needed, unlike silk_touch_drop. Spawn
eggs, /summon, and monster spawners are untouched by design.

Update README.md and AGENTS.md for the two-pack repo.
Add scripts/validate_packs.py to check manifest schema, UUID
validity/uniqueness across all packs, and spawn_rules structure.
Generalize CI's JSON/JS syntax checks to cover all tracked files
instead of only silk_touch_drop, so no_bat_spawn and future packs
are covered automatically.
package.json declared "packageManager": "bun@*", which setup-bun@v2
rejects outright ("Invalid argument not valid semver ('*' received)"),
failing both the test and pre-commit jobs before any steps run. Drop
the field so setup-bun falls back to installing latest bun.
The "increase-if-necessary value was missing its closing quote,
breaking YAML parsing entirely (check-yaml pre-commit hook and
pre-commit.ci both failed on it). Pre-existing bug from PR #11,
unrelated to the pack-validation changes in this branch.
@Ven0m0
Ven0m0 merged commit 4b53c8d into main Aug 16, 2026
8 of 10 checks passed
@Ven0m0
Ven0m0 deleted the claude/plan-md-implementation-oufb1h branch August 16, 2026 07:44
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