AI_Assistant/pcbjam
Watch
1
0
Fork
You've already forked pcbjam
 0
Code Issues Pull requests Projects Releases 7 Packages Wiki Activity Actions

fix: wrapInBoardEnvelope layer order

Browse source 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ADzCSeN3Q9DJL3YW2FZTXB
This commit is contained in:
Gergő Törcsvári 2026-07-13 09:13:27 +02:00
commit f07b9970d5
No known key found for this signature in database
GPG key ID: 8E75F2CDE64E5322
1 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
wasm/bindings/pcbnew_embind.cpp
View file

@ -419,7 +419,7 @@ std::string wrapInBoardEnvelope( BOARD& aBoard, const std::string& aItemSexpr )
std::string s = "(kicad_pcb (version " + std::to_string( SEXPR_BOARD_FILE_VERSION ) std::string s = "(kicad_pcb (version " + std::to_string( SEXPR_BOARD_FILE_VERSION )
+ ") (generator \"pcbnew\") (layers"; + ") (generator \"pcbnew\") (layers";
for( PCB_LAYER_ID id : aBoard.GetEnabledLayers().Seq() ) auto emit = [&]( PCB_LAYER_ID id )
{ {
const char* type = IsCopperLayer( id ) ? LAYER::ShowType( aBoard.GetLayerType( id ) ) const char* type = IsCopperLayer( id ) ? LAYER::ShowType( aBoard.GetLayerType( id ) )
: "user"; : "user";
@ -430,7 +430,17 @@ std::string wrapInBoardEnvelope( BOARD& aBoard, const std::string& aItemSexpr )
// parse threw "not in fixed layer hash" for any board with such layers. // parse threw "not in fixed layer hash" for any board with such layers.
s += " (" + std::to_string( (int) id ) + " \"" s += " (" + std::to_string( (int) id ) + " \""
+ std::string( LSET::Name( id ).utf8_str() ) + "\" " + type + ")"; + std::string( LSET::Name( id ).utf8_str() ) + "\" " + type + ")";
} };
// Copper first (front→back), then tech/user — the formatBoardLayers order.
// parseLayers stops counting copper at the first non-copper entry and then
// rejects the count (<2), so raw Seq() order (which interleaves copper and
// non-copper ids in the v9 numbering) fails on EVERY bare payload.
for( PCB_LAYER_ID id : aBoard.GetEnabledLayers().CuStack() )
emit( id );
for( PCB_LAYER_ID id : aBoard.GetEnabledLayers().TechAndUserUIOrder() )
emit( id );
s += ") " + aItemSexpr + ")"; s += ") " + aItemSexpr + ")";
return s; return s;