Skip to content

Fix B9 stock patch cfg errors#27

Open
Sean-Kenneth-Doherty wants to merge 1 commit into
zer0Kerbal:masterfrom
Sean-Kenneth-Doherty:fix-config-patching-errors
Open

Fix B9 stock patch cfg errors#27
Sean-Kenneth-Doherty wants to merge 1 commit into
zer0Kerbal:masterfrom
Sean-Kenneth-Doherty:fix-config-patching-errors

Conversation

@Sean-Kenneth-Doherty

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty commented May 17, 2026

Copy link
Copy Markdown

Summary

  • Close two unclosed VERSION blocks in both changelog copies.
  • Move StockSwitch/refType matching onto valid ModuleManager node selectors.
  • Delete temporary refVolume/refType helper values with value-delete syntax before adding subtypes.
  • Keep the description suffix patches while removing invalid value-level HAS filters.
  • Correct the KGEx subtype tankType names to match the tank types actually provided by KGEx.

Why

Issues #3 and #4 report ModuleManager/B9 patching errors. The affected cfg files mixed invalid value-level selectors with helper values used later by B9PartSwitch, and the KGEx optional subtypes referenced stale KGX... tank type ids. That can leave B9PartSwitch trying to load subtypes whose tank types do not exist.

Resolves #3
Resolves #4

Checks

  • git diff --check
  • Headless KSP 1.12.5.3190 load with ModuleManager 4.2.3, B9PartSwitch 2.21.0.4, KGEx 1.0.0.2, CommunityResourcePack 112.0.1, and NearFutureExploration 1.1.3 reached MAINMENU.
  • That load reported ModuleManager: 612 patches applied, no ModuleManager errors, and no B9PartSwitch/No tank type errors.
  • Generated cache spot-checks show StockSwitch modules using the corrected KGExSupport, KGExXenon, KGExOre, KGExOxidizer, and KGExH2Ow tank types.
  • Additional optional-stack load with SimpleConstruction 1:4.0.2.7 reached MAINMENU, reported ModuleManager: 743 patches applied, registered the KGEx and SimpleConstruction tank types, and had no ModuleManager or B9PartSwitch tank-loading errors.

Notes

  • I did not run an interactive editor/VAB smoke test.
  • The SimpleConstruction optional-stack log still contains one non-B9 ExperienceSystemConfig duplicate Engineer effect line; it did not prevent the load reaching MAINMENU and is separate from the B9PartSwitch tank-type failures fixed here.

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty force-pushed the fix-config-patching-errors branch from 309ca9f to fa68474 Compare May 18, 2026 18:50
@Sean-Kenneth-Doherty Sean-Kenneth-Doherty marked this pull request as ready for review May 18, 2026 18:58
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.

[Bug 🐞]: B9 reports errors in patching [Bug 🐞]: MM reports errors in patching

1 participant