Skip to content

refactor(capture): extract surface sweep record builders#609

Merged
w1ne merged 1 commit into
developfrom
codex/surface-sweep-record-builders
Jul 9, 2026
Merged

refactor(capture): extract surface sweep record builders#609
w1ne merged 1 commit into
developfrom
codex/surface-sweep-record-builders

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • extract variableSweep and surfaceFromBoundary record construction into src/modeling/capture/surfaceSweepRecords.ts
  • keep CaptureSession responsible for id allocation, session lookup, OCCT endpoint evaluation, and proxy return wiring
  • add direct record-shape, fallback, unresolved-metadata, and import-boundary regression coverage

Verification

  • npm test -- tests/unit/capture/surfaceSweepRecords.test.ts tests/unit/capture/variableSweep.test.ts tests/unit/capture/surfaceFromBoundary.test.ts tests/unit/kernel/architectureBoundary.test.ts
  • npm test -- tests/integration/variableSweepEndToEnd.test.ts tests/unit/backends/occt/variableSweepLowerer.test.ts tests/unit/capture/surfaceProxy.test.ts
  • npm run typecheck
  • npx eslint src/modeling/capture/captureSession.ts src/modeling/capture/surfaceSweepRecords.ts tests/unit/capture/surfaceSweepRecords.test.ts
  • git diff --cached --check

@w1ne w1ne merged commit 4832a5b into develop Jul 9, 2026
13 checks passed
@w1ne w1ne deleted the codex/surface-sweep-record-builders branch July 9, 2026 19:04
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.

1 participant