Skip to content

NURBS Slice F: curved patch trim and true split#530

Merged
w1ne merged 8 commits into
developfrom
feat/nurbs-slice-f-curved-patch-trim
Jun 25, 2026
Merged

NURBS Slice F: curved patch trim and true split#530
w1ne merged 8 commits into
developfrom
feat/nurbs-slice-f-curved-patch-trim

Conversation

@w1ne

@w1ne w1ne commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Rebuilds the forked OCCT wasm dependency to kcad-v0.24.0 with BRepFeat_SplitShape available, and verifies the installed binary exposes it.
  • Replaces planar-only surface trim lowering with BRepAlgoAPI_Section + BRepFeat_SplitShape imprinting for clean curved patch trims.
  • Changes Surface.split(by) to return both halves as [Surface, Surface], updates docs/MCP/list_api/skills, and bumps version to 0.15.0.

Closes #528

Verification

  • node wasm probe: BRepFeat_SplitShape present; BRepAlgoAPI_Section present
  • npx vitest run tests/unit/intent/surfaceTrim.test.ts
  • npx vitest run tests/unit/backends/occt/surfaceTrim.test.ts
  • npx vitest run tests/integration/mcp/listApi.driftSentinel.test.ts tests/integration/mcp/finishSurface.test.ts tests/unit/cli/skill.test.ts tests/unit/diagnostics/codes.test.ts tests/unit/diagnostics/emittedCodesAreCatalogued.test.ts
  • npm run typecheck
  • npm run lint (exit 0; existing unused-disable warnings remain)

@w1ne w1ne enabled auto-merge June 24, 2026 21:37
@w1ne w1ne merged commit a001519 into develop Jun 25, 2026
17 checks passed
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.

NURBS Slice F: curved patch trim and true split

1 participant