Add An Autofix CLI For Invalid Format Versions #15

nathan · 2026-09-08T16:22:31Z

nathan commented

2026-09-08 16:22:31 +00:00

Summary

Adds npm run fix:formatVersions. It scans the same assets as npm run validate:formatVersions and bumps outdated format_version values in place.

Groups outdated files by category (models, animations, animation controllers, render controllers, particles, entities, attachables, sounds).
Prints each path with its current version and the required version.
Asks once per category before writing. Declining a category leaves those files unchanged.
Replaces only the first format_version string, so the rest of the file stays as it is.
After accepting every category, npm run validate:formatVersions should pass. Files with no format_version, or a legacy geometry.* root, are skipped and still need a manual edit.

Test plan

npm run validate:formatVersions is green on this branch
Lower one animation format_version, run npm run fix:formatVersions, accept that category, confirm the string bumped and validate is green again
Decline a category and confirm those files stay unchanged

## Summary Adds `npm run fix:formatVersions`. It scans the same assets as `npm run validate:formatVersions` and bumps outdated `format_version` values in place. - Groups outdated files by category (models, animations, animation controllers, render controllers, particles, entities, attachables, sounds). - Prints each path with its current version and the required version. - Asks once per category before writing. Declining a category leaves those files unchanged. - Replaces only the first `format_version` string, so the rest of the file stays as it is. - After accepting every category, `npm run validate:formatVersions` should pass. Files with no `format_version`, or a legacy `geometry.*` root, are skipped and still need a manual edit. ## Test plan - [ ] `npm run validate:formatVersions` is green on this branch - [ ] Lower one animation `format_version`, run `npm run fix:formatVersions`, accept that category, confirm the string bumped and validate is green again - [ ] Decline a category and confirm those files stay unchanged

nathan added 1 commit

2026-09-08 16:22:31 +00:00

Init

Auto Assign / run (pull_request_target) Successful in 2s

Details

Model Preview / model-preview (pull_request) Failing after 12s

Details

CI / build-and-test (20.x) (pull_request) Successful in 30s

Details

3f3ad944c9

smell-of-curry was assigned by forgejo-actions

2026-09-08 16:22:32 +00:00

bot requested changes

2026-09-08 17:10:02 +00:00

Dismissed

bot left a comment

Review of head 3f3ad944c9fc2 — new fix:formatVersions CLI + first-run asset bumps. No prior reviews. mergeable=true vs main f3dee531. Auth as bot. No open same-repo issues to link.

Blocker: collectOutdatedVersionEntries stores stripSubpackPrefix(relPath) as relPath. SCAN_DIRS includes subpacks/, so outdated subpack assets get written to the wrong place (root counterpart / miss) when fix:formatVersions joins process.cwd() + that stripped path. Keep the full pack-relative path for IO; strip only for category resolution / display if needed.

Scope: title is CLI-focused, but the PR also lands ~25 format_version bumps. Same charm/hyde/rotom/gymleader/entity bumps already sit on open res#8 head 36667487 (identical catching_charm blob). Prefer CLI-only here and keep the data migration on one PR, or state clearly that #15 owns the bumps and strip them from #8 to avoid double-merge churn.

Otherwise the validate refactor (getRequiredInfo / walkValidatedFiles / JSDoc) looks solid, @inquirer/prompts is already a dependency, and charm → 1.16.100 matches attachableClient (c.is_first_person). Empty PR body / Init commit — please describe intended usage + whether validate is green after the bumps.

Nit: async main() is invoked without .catch (unhandled rejection on prompt/IO failure).

Review of head `3f3ad944c9fc2` — new `fix:formatVersions` CLI + first-run asset bumps. No prior reviews. mergeable=true vs main `f3dee531`. Auth as bot. No open same-repo issues to link. **Blocker:** `collectOutdatedVersionEntries` stores `stripSubpackPrefix(relPath)` as `relPath`. `SCAN_DIRS` includes `subpacks/`, so outdated subpack assets get written to the wrong place (root counterpart / miss) when `fix:formatVersions` joins `process.cwd()` + that stripped path. Keep the full pack-relative path for IO; strip only for category resolution / display if needed. **Scope:** title is CLI-focused, but the PR also lands ~25 `format_version` bumps. Same charm/hyde/rotom/gymleader/entity bumps already sit on open [res#8](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/pulls/8) head `36667487` (identical `catching_charm` blob). Prefer CLI-only here and keep the data migration on one PR, or state clearly that #15 owns the bumps and strip them from #8 to avoid double-merge churn. Otherwise the validate refactor (`getRequiredInfo` / `walkValidatedFiles` / JSDoc) looks solid, `@inquirer/prompts` is already a dependency, and charm → `1.16.100` matches `attachableClient` (`c.is_first_person`). Empty PR body / `Init` commit — please describe intended usage + whether validate is green after the bumps. Nit: async `main()` is invoked without `.catch` (unhandled rejection on prompt/IO failure).

scripts/fixFormatVersions.ts

					
				@ -0,0 +90,4 @@

				    if (!shouldUpdate) continue;

				    for (const entry of list) {

				      const absPath = path.join(process.cwd(), entry.relPath);

Uses entry.relPath for the write target. Once collectOutdatedVersionEntries returns the real pack-relative path (including any subpacks/... prefix), this join is correct; with the current stripped path it can create/update the wrong file.

Uses `entry.relPath` for the write target. Once `collectOutdatedVersionEntries` returns the real pack-relative path (including any `subpacks/...` prefix), this join is correct; with the current stripped path it can create/update the wrong file.

nathan marked this conversation as resolved

scripts/fixFormatVersions.ts Outdated

					
				@ -0,0 +98,4 @@

				  console.log(`\n✅ Updated ${updatedCount} file(s).`);

				}

				if (require.main === module) main();

Nit: main() is async — prefer main().catch((err) => { console.error(err); process.exit(1); }) so prompt/IO failures do not become unhandled rejections.

Nit: `main()` is async — prefer `main().catch((err) => { console.error(err); process.exit(1); })` so prompt/IO failures do not become unhandled rejections.

nathan marked this conversation as resolved

scripts/validateFormatVersions.ts

					
				@ -198,0 +235,4 @@

				    const version = data["format_version"];

				    if (typeof version !== "string" || isAtLeast(version, info.required)) return;

				    entries.push({ relPath: rel, category: info.category, current: version, required: info.required });

Blocker: rel is stripSubpackPrefix(...), then stored as relPath. Subpack files under subpacks/<name>/... (also walked via SCAN_DIRS) will report/fix the wrong path. Store the original pack-relative relPath for IO; use the stripped form only when resolving category.

**Blocker:** `rel` is `stripSubpackPrefix(...)`, then stored as `relPath`. Subpack files under `subpacks/<name>/...` (also walked via `SCAN_DIRS`) will report/fix the wrong path. Store the original pack-relative `relPath` for IO; use the stripped form only when resolving category.

nathan marked this conversation as resolved

nathan added 1 commit

2026-09-08 18:44:15 +00:00

Resolve Comments And Nitpics

Model Preview / model-preview (pull_request) Failing after 13s

Details

CI / build-and-test (20.x) (pull_request) Successful in 29s

Details

3095b489f5

bot requested changes

2026-09-08 19:38:02 +00:00

Dismissed

bot left a comment

Re-review of head 3095b489 (was 3f3ad944). Nathan resolved the three prior inlines (subpack relPath write bug + .catch). CI build-and-test green on this head (run 160 / 29s). Model Preview red (run 161) — not treating as the merge gate. No open same-repo issues to link. Auth as bot.

Fixed: collectOutdatedVersionEntries now stores the real pack-relative relPath (JSDoc matches); main().catch(...) is in place.

Still open — scope / double-merge: title is CLI-focused, but ~25 format_version bumps remain and still overlap open res#8 (charms/hyde/rotom/gymleader paths). Pick one and write it in the PR body:

#15 owns the bumps (reasonable — these are exactly what is failing validate:formatVersions on main) → strip the identical format_version-only churn from #8 before either merges; or
CLI-only → drop the asset bumps from this PR and leave the migration on one data PR.

Empty PR body still — please describe npm run fix:formatVersions usage + that validate is green after the bumps.

Nit (non-blocking): confirm prompt missing ? (Do you want to update them).

Re-review of head `3095b489` (was `3f3ad944`). Nathan resolved the three prior inlines (subpack `relPath` write bug + `.catch`). CI `build-and-test` green on this head (run 160 / 29s). Model Preview red (run 161) — not treating as the merge gate. No open same-repo issues to link. Auth as bot. **Fixed:** `collectOutdatedVersionEntries` now stores the real pack-relative `relPath` (JSDoc matches); `main().catch(...)` is in place. **Still open — scope / double-merge:** title is CLI-focused, but ~25 `format_version` bumps remain and still overlap open [res#8](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/pulls/8) (charms/hyde/rotom/gymleader paths). Pick one and write it in the PR body: 1. **#15 owns the bumps** (reasonable — these are exactly what is failing `validate:formatVersions` on main) → strip the identical format_version-only churn from #8 before either merges; or 2. **CLI-only** → drop the asset bumps from this PR and leave the migration on one data PR. Empty PR body still — please describe `npm run fix:formatVersions` usage + that validate is green after the bumps. Nit (non-blocking): confirm prompt missing `?` (`Do you want to update them`).

package.json

					
				@ -6,6 +6,7 @@

				  "scripts": {

				    "test": "jest",

				    "validate:formatVersions": "npx ts-node --project tsconfig.json scripts/validateFormatVersions.ts",

				    "fix:formatVersions": "npx ts-node --project tsconfig.json scripts/fixFormatVersions.ts",

Scope still open: fix:formatVersions is the feature, but this PR also lands ~25 format_version asset bumps that overlap res#8. Either declare in the PR body that #15 owns those bumps and strip the same format_version-only churn from #8, or drop the bumps here and keep this CLI-only.

**Scope still open:** `fix:formatVersions` is the feature, but this PR also lands ~25 `format_version` asset bumps that overlap [res#8](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/pulls/8). Either declare in the PR body that #15 owns those bumps and strip the same format_version-only churn from #8, or drop the bumps here and keep this CLI-only.

smell-of-curry added 1 commit

2026-10-01 18:55:21 +00:00

Merge branch 'main' into feature/implement-autofix-invalid-format-versions

CI / build-and-test (20.x) (pull_request) Successful in 30s

Details

Model Preview / model-preview (pull_request) Has been cancelled

Details

f09d33516d

Co-authored-by: Cursor <cursoragent@cursor.com>

bot requested changes

2026-10-01 19:03:18 +00:00

Dismissed

bot left a comment

Re-review of head f09d33516d5830f8bc832361827c96dfd9c80096 (was 3095b489 — merge main into feature). Auth as bot. mergeable=true vs main 5a2ff211. CI build-and-test green (run 331 / ~30s). Model Preview still pending — not treating as the merge gate. No open same-repo issues to link.

Prior blockers cleared:

collectOutdatedVersionEntries still stores the real pack-relative relPath (incl. subpacks/…) for IO; strip only for category resolution.
main().catch(...) still in place.
Overlap with res#8 is gone (merged 2026-09-11). Main 5a2ff211 already shipped the broad format_version migration, so the old “who owns the bumps” fight is moot.

Still open — drop leftover asset churn: the six animation paths still in the diff (wrist_charm, hyde, dragon/electric/ghost/psychic_gymleader) are whitespace-only (spaces→tabs / blank-line reshuffle). Semantic JSON is unchanged and format_version stays at already-required 1.10.0 — they are not autofix output that moves versions. Title is CLI-focused; please revert those six files so this PR is package.json + scripts/fixFormatVersions.ts + scripts/validateFormatVersions.ts only.

Still open — empty PR body: please describe npm run fix:formatVersions usage (interactive per-category confirm) and that validate is expected green after a fix run.

Nit (non-blocking): confirm prompt still missing ? (Do you want to update them).

CLI refactor itself looks solid once the indent noise is gone.

Re-review of head `f09d33516d5830f8bc832361827c96dfd9c80096` (was `3095b489` — merge main into feature). Auth as bot. mergeable=true vs main `5a2ff211`. CI `build-and-test` green (run 331 / ~30s). Model Preview still pending — not treating as the merge gate. No open same-repo issues to link. **Prior blockers cleared:** - `collectOutdatedVersionEntries` still stores the real pack-relative `relPath` (incl. `subpacks/…`) for IO; strip only for category resolution. - `main().catch(...)` still in place. - Overlap with [res#8](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/pulls/8) is gone (merged 2026-09-11). Main `5a2ff211` already shipped the broad format_version migration, so the old “who owns the bumps” fight is moot. **Still open — drop leftover asset churn:** the six animation paths still in the diff (`wrist_charm`, `hyde`, `dragon/electric/ghost/psychic_gymleader`) are **whitespace-only** (spaces→tabs / blank-line reshuffle). Semantic JSON is unchanged and `format_version` stays at already-required `1.10.0` — they are not autofix output that moves versions. Title is CLI-focused; please revert those six files so this PR is `package.json` + `scripts/fixFormatVersions.ts` + `scripts/validateFormatVersions.ts` only. **Still open — empty PR body:** please describe `npm run fix:formatVersions` usage (interactive per-category confirm) and that validate is expected green after a fix run. Nit (non-blocking): confirm prompt still missing `?` (`Do you want to update them`). CLI refactor itself looks solid once the indent noise is gone.

scripts/fixFormatVersions.ts

					
				@ -0,0 +84,4 @@

				    }

				    const shouldUpdate = await confirm({

				      message: "Do you want to update them",

Nit: confirm message is missing a trailing ? (Do you want to update them → Do you want to update them?).

Nit: confirm message is missing a trailing `?` (`Do you want to update them` → `Do you want to update them?`).

bot requested changes

2026-10-01 19:03:18 +00:00

Dismissed

bot left a comment

REQUEST_CHANGES on head f09d33516d5830f8bc832361827c96dfd9c80096 (merge of main 5a2ff211 into feature/implement-autofix-invalid-format-versions). mergeable=true. CI gate build-and-test success (run 331 / ~30s). Model Preview run 332 pending — non-gate. Open same-repo issues: 0 (no Fixes/Could Fix). Auth as bot. No prior bot review on this tip.

Cleared since 3095b489:

Subpack relPath IO + main().catch still correct.
Scope overlap with res#8 is gone — #8 is merged; main 5a2ff211 already raised the stale format_versions.

Still open

Drop leftover animation churn — the 6 animation JSON files (wrist_charm, hyde, *_gymleader) are semantically identical to main (format_version already 1.10.0; JSON equal). Diff is tabs/whitespace/wrap only, not what bumpFormatVersion does. Title is CLI-focused; keep this PR scripts + package.json only.
Empty PR body — still blank. Please describe npm run fix:formatVersions (category prompts, safe in-place bump) and that validate is green on tip.

CLI/validateFormatVersions refactor looks good (getRequiredInfo, walkValidatedFiles, JSDoc, OutdatedVersionEntry.relPath docs).

Nit (non-blocking, still present): confirm prompt Do you want to update them missing ?.

Next: strip the 6 no-op animation files + fill PR body → re-review for approve.

**REQUEST_CHANGES** on head `f09d33516d5830f8bc832361827c96dfd9c80096` (merge of main `5a2ff211` into `feature/implement-autofix-invalid-format-versions`). mergeable=true. CI gate `build-and-test` **success** (run 331 / ~30s). Model Preview run 332 pending — non-gate. Open same-repo issues: 0 (no Fixes/Could Fix). Auth as bot. No prior bot review on this tip. **Cleared since `3095b489`:** - Subpack `relPath` IO + `main().catch` still correct. - Scope overlap with [res#8](https://git.pokebedrock.com/pokebedrock/pokebedrock-res/pulls/8) is gone — #8 is merged; main `5a2ff211` already raised the stale `format_version`s. **Still open** 1. **Drop leftover animation churn** — the 6 animation JSON files (`wrist_charm`, `hyde`, `*_gymleader`) are semantically identical to main (`format_version` already `1.10.0`; JSON equal). Diff is tabs/whitespace/wrap only, not what `bumpFormatVersion` does. Title is CLI-focused; keep this PR scripts + `package.json` only. 2. **Empty PR body** — still blank. Please describe `npm run fix:formatVersions` (category prompts, safe in-place bump) and that validate is green on tip. CLI/`validateFormatVersions` refactor looks good (`getRequiredInfo`, `walkValidatedFiles`, JSDoc, `OutdatedVersionEntry.relPath` docs). Nit (non-blocking, still present): confirm prompt `Do you want to update them` missing `?`. **Next:** strip the 6 no-op animation files + fill PR body → re-review for approve.

animations/attachables/wrist_charm.json

					
				@ -23,3 +2,1 @@

				    }

				  }

				}

					"format_version": "1.10.0",

Scope: tip vs main is whitespace-only (tabs/indent); format_version is already 1.10.0 on both and JSON content is identical. Same for hyde + the four *_gymleader animation files in this PR. Please revert these 6 asset files so the diff stays CLI-only (scripts/* + package.json).

**Scope:** tip vs main is whitespace-only (tabs/indent); `format_version` is already `1.10.0` on both and JSON content is identical. Same for `hyde` + the four `*_gymleader` animation files in this PR. Please revert these 6 asset files so the diff stays CLI-only (`scripts/*` + `package.json`).

smell-of-curry added 1 commit

2026-10-01 19:03:35 +00:00

Point model preview at the Forgejo copy of the action.

CI / build-and-test (20.x) (pull_request) Successful in 31s

Details

Model Preview / model-preview (pull_request) Successful in 1m37s

Details

385db876f4

bot requested changes

2026-10-01 19:09:31 +00:00

Dismissed

bot left a comment

REQUEST_CHANGES on head 385db876f45f63daf8db704e361f298aa64cb562 (was f09d3351 — sole new commit: point model-preview at Forgejo pokebedrock/mc-model-preview@b10a8ffc). mergeable=true vs main 5a2ff211 / merge-base. Auth as bot. Open same-repo issues: 0 — no Fixes/Could Fix. CI on this tip still pending (build-and-test run 347 + Model Preview run 348) at review time; prior tip had green build-and-test.

New since last review — OK: .github/workflows/model-preview.yml correctly leaves GitHub smell-of-curry/mc-model-preview@latest for a Forgejo-pinned https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffc…. Chrome install step unchanged. Non-blocking on its own.

Still open (unchanged from REQUEST_CHANGES on f09d3351):

Drop leftover animation churn — the 6 animation JSON files (wrist_charm, hyde, dragon/electric/ghost/psychic_gymleader) remain json_equal to main with format_version already 1.10.0. Diff is tabs/whitespace/wrap only, not bumpFormatVersion output. Title is CLI-focused; keep this PR to package.json + scripts/fixFormatVersions.ts + scripts/validateFormatVersions.ts (+ the model-preview pin is fine to keep).
Empty PR body — still blank. Please describe npm run fix:formatVersions (interactive per-category confirm, in-place bump) and that validate is expected green after a fix run.

CLI/validateFormatVersions refactor still looks solid (relPath pack-relative for IO, main().catch, JSDoc).

Nit (non-blocking, still present): confirm prompt Do you want to update them missing ?.

Next: strip the 6 no-op animation files + fill PR body → re-review for approve.

**REQUEST_CHANGES** on head `385db876f45f63daf8db704e361f298aa64cb562` (was `f09d3351` — sole new commit: point model-preview at Forgejo `pokebedrock/mc-model-preview@b10a8ffc`). mergeable=true vs main `5a2ff211` / merge-base. Auth as bot. Open same-repo issues: 0 — no Fixes/Could Fix. CI on this tip still pending (`build-and-test` run 347 + Model Preview run 348) at review time; prior tip had green `build-and-test`. **New since last review — OK:** `.github/workflows/model-preview.yml` correctly leaves GitHub `smell-of-curry/mc-model-preview@latest` for a Forgejo-pinned `https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffc…`. Chrome install step unchanged. Non-blocking on its own. **Still open (unchanged from REQUEST_CHANGES on `f09d3351`):** 1. **Drop leftover animation churn** — the 6 animation JSON files (`wrist_charm`, `hyde`, `dragon/electric/ghost/psychic_gymleader`) remain `json_equal` to main with `format_version` already `1.10.0`. Diff is tabs/whitespace/wrap only, not `bumpFormatVersion` output. Title is CLI-focused; keep this PR to `package.json` + `scripts/fixFormatVersions.ts` + `scripts/validateFormatVersions.ts` (+ the model-preview pin is fine to keep). 2. **Empty PR body** — still blank. Please describe `npm run fix:formatVersions` (interactive per-category confirm, in-place bump) and that validate is expected green after a fix run. CLI/`validateFormatVersions` refactor still looks solid (`relPath` pack-relative for IO, `main().catch`, JSDoc). Nit (non-blocking, still present): confirm prompt `Do you want to update them` missing `?`. **Next:** strip the 6 no-op animation files + fill PR body → re-review for approve.

forgejo-actions commented

2026-10-01 19:10:19 +00:00

Minecraft Model Preview (`385db87`)

Model Changes

Entity	Before	After
No renderable model changes detected

## Minecraft Model Preview (385db87) ### Model Changes | Entity | Before | After | |--------|--------|-------| | _No renderable model changes detected_ | | |

bot requested changes

2026-10-01 19:10:44 +00:00

bot left a comment

REQUEST_CHANGES on head 385db876f45f63daf8db704e361f298aa64cb562 (was f09d3351 — Point model preview at the Forgejo copy of the action.). Auth as bot. mergeable=true vs moving main. CI build-and-test + Model Preview still pending on this tip — not treating as green yet. Open same-repo issues: 0 (no Fixes/Could Fix).

Tip delta vs last review: only .github/workflows/model-preview.yml — uses: now pins https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffc… (matches current main). Fine; not a blocker.

Still open (unchanged since REQUEST_CHANGES#2950 @ f09d3351)

Drop leftover animation churn — the 6 animation JSON files (wrist_charm, hyde, dragon/electric/ghost/psychic_gymleader) remain JSON-equal to main (format_version already 1.10.0). Diff is whitespace/indent only, not autofix output. Keep this PR to package.json + scripts/fixFormatVersions.ts + scripts/validateFormatVersions.ts (+ the workflow pin is OK).
Empty PR body — still blank. Please describe npm run fix:formatVersions (per-category confirm, in-place bump) and that validate is expected green after a fix run.

CLI/validateFormatVersions refactor still looks solid (collectOutdatedVersionEntries keeps full pack-relative relPath, main().catch, JSDoc).

Nit (non-blocking): confirm prompt still Do you want to update them (missing ?).

Next: strip the 6 no-op animation files + fill PR body → re-review for approve once CI settles.

**REQUEST_CHANGES** on head `385db876f45f63daf8db704e361f298aa64cb562` (was `f09d3351` — *Point model preview at the Forgejo copy of the action.*). Auth as bot. mergeable=true vs moving main. CI `build-and-test` + Model Preview still **pending** on this tip — not treating as green yet. Open same-repo issues: 0 (no Fixes/Could Fix). **Tip delta vs last review:** only `.github/workflows/model-preview.yml` — `uses:` now pins `https://git.pokebedrock.com/pokebedrock/mc-model-preview@b10a8ffc…` (matches current main). Fine; not a blocker. **Still open (unchanged since REQUEST_CHANGES#2950 @ `f09d3351`)** 1. **Drop leftover animation churn** — the 6 animation JSON files (`wrist_charm`, `hyde`, `dragon/electric/ghost/psychic_gymleader`) remain **JSON-equal to main** (`format_version` already `1.10.0`). Diff is whitespace/indent only, not autofix output. Keep this PR to `package.json` + `scripts/fixFormatVersions.ts` + `scripts/validateFormatVersions.ts` (+ the workflow pin is OK). 2. **Empty PR body** — still blank. Please describe `npm run fix:formatVersions` (per-category confirm, in-place bump) and that validate is expected green after a fix run. CLI/`validateFormatVersions` refactor still looks solid (`collectOutdatedVersionEntries` keeps full pack-relative `relPath`, `main().catch`, JSDoc). Nit (non-blocking): confirm prompt still `Do you want to update them` (missing `?`). **Next:** strip the 6 no-op animation files + fill PR body → re-review for approve once CI settles.

forgejo-actions commented

2026-10-13 13:14:00 +00:00

Minecraft Model Preview (`0cb915b`)

Model Changes

Entity	Before	After
No renderable model changes detected

## Minecraft Model Preview (0cb915b) ### Model Changes | Entity | Before | After | |--------|--------|-------| | _No renderable model changes detected_ | | |

smell-of-curry referenced this pull request

2026-10-13 13:15:33 +00:00

Add an autofix CLI for invalid format versions #22

smell-of-curry commented

2026-10-13 13:15:33 +00:00

Superseded by #22. The six whitespace-only animation files are gone, and the CLI description is on the new PR. This branch's diff stayed rooted at 5a2ff211, so Forgejo still listed files that are already on main. Closing so that one is the review.

Superseded by #22. The six whitespace-only animation files are gone, and the CLI description is on the new PR. This branch's diff stayed rooted at `5a2ff211`, so Forgejo still listed files that are already on `main`. Closing so that one is the review.

smell-of-curry closed this pull request

2026-10-13 13:15:34 +00:00

smell-of-curry referenced this pull request from a commit

2026-10-15 01:43:47 +00:00

Add an autofix CLI for invalid format versions (#22)

CI / build-and-test (20.x) (pull_request) Successful in 31s

Details

Model Preview / model-preview (pull_request) Successful in 1m37s

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

No assignees

4 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

pokebedrock/pokebedrock-res!15

Rows
Columns