Skip to content

fix: drop skins with no model and fix the cart's hull smoke bone - #79

Merged
XxFran10xX merged 1 commit into
mainfrom
fix/remove-unbuilt-skins
Oct 22, 2026
Merged

XxFran10xX merged 1 commit into
mainfrom
fix/remove-unbuilt-skins

Conversation

@XxFran10xX

Copy link
Copy Markdown
Contributor

Summary

Config-only cleanup of two warnings that Main logs every day.

  • Skins with no model. biplane, monoplane, cloudskimmer and gyrobomber list 11 skins (Prism, Norain, Zerratoris, Oseni) that have no ModelEngine blueprint on Main or Dev. Each start logs Skin '…' references missing ModelEngine model '…', and SkinHandler refuses to switch to them, so players can never use them. This removes those 11 entries. The base skin and the 8 built variants (Black, Yellow, Purple Imvata, Revenor, Domenia, Sabarissa, Brown, Pirate) stay.
  • Wooden cart hull smoke. The hull's vfx lists vfx1, but wooden_cart.bbmodel only has vfx (front, harness), vfx2 (middle) and vfx3 (rear). Every cart load logged wooden_cart has no bone called vfx1 (22 times on Main today), and that hull smoke point was never shown. It now uses vfx3.

No code changes. Saved vehicles are unaffected: none of Main's 116 stored vehicles uses a removed skin.

Testing

Bot test on TFMCDev01 (~/vf-skins-test/test.js). Dev's configs matched Main's before the test.

  • Old config: /vf spawn wooden_cart logged has no bone called vfx1.
  • Swapped in the edited configs and ran vf reload: no references missing ModelEngine model lines and no VehicleFramework errors.
  • New config: /vf spawn wooden_cart logged no bone warnings. A biplane also spawned cleanly.

Companion changes: the same config edits in ServerAssets, and the wiki skin list in TFMCWeb.

🤖 Generated with Claude Code

Eleven plane and airship skins (Prism, Norain, Zerratoris and Oseni) are
listed under skins: but no ModelEngine blueprint exists for them, so every
start logs "references missing ModelEngine model" and the skins can never
be picked. Remove those entries; the built skins stay.

The wooden cart's hull lists a vfx1 bone the model does not have (it has
vfx, vfx2 and vfx3), which logged "has no bone called vfx1" on every cart
load and left one smoke point unused. Use vfx3, the rear bone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 157645dc-eb55-41f9-bede-277d1a2a9fcf
📥 Commits

Reviewing files that changed from the base of the PR and between 10ba3f9 and 648f540.

📒 Files selected for processing: 5
  • src/main/resources/vehicles/biplane.yml
  • src/main/resources/vehicles/cloudskimmer.yml
  • src/main/resources/vehicles/gyrobomber.yml
  • src/main/resources/vehicles/monoplane.yml
  • src/main/resources/vehicles/wooden_cart.yml
💤 Files with no reviewable changes: 4
  • src/main/resources/vehicles/gyrobomber.yml
  • src/main/resources/vehicles/biplane.yml
  • src/main/resources/vehicles/monoplane.yml
  • src/main/resources/vehicles/cloudskimmer.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Changes
    • Removed the Prism, Norain, Zerratoris and Oseni skins from the Biplane and Monoplane, and the Prism and Norain skins from the Cloudskimmer.
    • Removed the Prism skin from the Gyrobomber.
    • Updated the wooden cart’s hull visual effects.

The aircraft configurations no longer include the listed skin variants. The wooden cart hull VFX list now uses vfx3 instead of vfx1.

Changes: Aircraft skin entries

Layer / File(s) Summary
Remove aircraft skin entries
src/main/resources/vehicles/biplane.yml, src/main/resources/vehicles/cloudskimmer.yml, src/main/resources/vehicles/gyrobomber.yml, src/main/resources/vehicles/monoplane.yml
Removed the specified skin entries and their associated names and model identifiers. Other skin entries remain unchanged.

Wooden cart VFX

Layer / File(s) Summary
Change hull VFX selection
src/main/resources/vehicles/wooden_cart.yml
The hull VFX list now uses vfx3 instead of vfx1. vfx2 remains.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix

Suggested reviewers: carolinebondhus

Merge Risk: ⚪ Minimal · up to 648f5

No actionable merge-blocking issue is established. The removed skins have no identified repository consumers; the wooden cart bone change remains unconfirmed from the available files.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the size:M 30-99 effective changed lines (tests excluded in mixed PRs). label Oct 22, 2026
@XxFran10xX

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@XxFran10xX
XxFran10xX merged commit 7056fcd into main Oct 22, 2026
3 checks passed
@XxFran10xX
XxFran10xX deleted the fix/remove-unbuilt-skins branch October 22, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 effective changed lines (tests excluded in mixed PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant