Skip to content

refactor(capture): extract authoring record builders#608

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

refactor(capture): extract authoring record builders#608
w1ne merged 1 commit into
developfrom
codex/authoring-record-builders

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • move dfmSpec, curve3d, embossText, and projectCurve record construction out of CaptureSession
  • keep face selector resolution in CaptureSession while authoringFeatureRecords stays a record-builder leaf
  • add byte-stable capture record coverage and helper boundary assertions

Verification

  • npm test -- tests/unit/capture/authoringFeatureRecords.test.ts tests/unit/capture/dfmSpec.test.ts tests/unit/capture/nurbsCurve.test.ts tests/unit/capture/embossText.test.ts tests/unit/capture/projectCurve.test.ts tests/unit/capture/shapeEmbossProjectProxy.test.ts
  • npm run typecheck
  • npx eslint src/modeling/capture/captureSession.ts src/modeling/capture/authoringFeatureRecords.ts tests/unit/capture/authoringFeatureRecords.test.ts
  • npm test -- tests/unit/kernel/architectureBoundary.test.ts
  • git diff --check

Subagent review: behavior-loss review found no issues; architecture review found one helper/proxy type dependency, fixed and re-reviewed clean.

@w1ne w1ne merged commit e40d9e8 into develop Jul 9, 2026
13 checks passed
@w1ne w1ne deleted the codex/authoring-record-builders branch July 9, 2026 18:41
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