Skip to content

Remove SlotFieldPublicSchema, which nothing parses with - #386

Merged
richshaw merged 1 commit into
mainfrom
claude/relaxed-fermat-38gbdp
Oct 4, 2026
Merged

richshaw merged 1 commit into
mainfrom
claude/relaxed-fermat-38gbdp

Conversation

@richshaw

@richshaw richshaw commented Oct 4, 2026

Copy link
Copy Markdown
Owner

SlotFieldPublicSchema in src/schemas/slot-fields.ts described a field paired with one slot's value (ref, label, fieldType, value, config). No service, route, MCP tool or component builds or parses that shape. Only its own two tests imported it, which is why knip didn't flag it.

This removes the schema and those tests. RefSchema, FIELD_TYPES and SlotFieldConfigSchema are still used by the input schemas.

Checks

  • pnpm lint, pnpm typecheck, pnpm knip: clean
  • pnpm test: 93 files, 1036 tests passed

No UI changes, so there are no screenshots.

🤖 Generated with Claude Code

https://claude.ai/code/session_012V2XKSqiFuL8TRtixdkDF7


Generated by Claude Code

The schema described a field paired with a slot's value, a shape no
service, route or tool builds. Only its own tests used it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012V2XKSqiFuL8TRtixdkDF7
@richshaw
richshaw merged commit 2a74742 into main Oct 4, 2026
5 checks passed
@richshaw
richshaw deleted the claude/relaxed-fermat-38gbdp branch October 4, 2026 16:09
richshaw pushed a commit that referenced this pull request Oct 4, 2026
Brings in #382, #386 and #388 (edit-token ops metered and read in one place).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0159b8Nfy3w7F8BVwvn37Fc3
richshaw pushed a commit that referenced this pull request Oct 4, 2026
Brings in main (#382, #386, #388) by way of #380.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0159b8Nfy3w7F8BVwvn37Fc3
richshaw pushed a commit that referenced this pull request Oct 4, 2026
Brings in #382, #386 and #388 (edit-token ops metered and read in one place).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0159b8Nfy3w7F8BVwvn37Fc3
richshaw pushed a commit that referenced this pull request Oct 4, 2026
Brings in main (#382, #386, #388) by way of #379; route.ts takes authorizeTokenOp and keeps the move's edit link and cookie swap.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0159b8Nfy3w7F8BVwvn37Fc3
richshaw pushed a commit that referenced this pull request Oct 4, 2026
Brings in main (#382, #386, #388) by way of #379 and #381.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0159b8Nfy3w7F8BVwvn37Fc3
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.

2 participants