Skip to content

refactor: remove unused exports#44

Merged
benvinegar merged 1 commit into
mainfrom
cleanup/remove-unused-exports
Jun 17, 2026
Merged

refactor: remove unused exports#44
benvinegar merged 1 commit into
mainfrom
cleanup/remove-unused-exports

Conversation

@benvinegar

@benvinegar benvinegar commented Jun 17, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove or privatize unused exports reported by knip
  • Drop the unused firstHtml helper and obsolete strict surface-parts aggregate schema
  • Keep internal MCP schemas, surface-page constants, viewer state setters, and sideshow-term palette private

Validation

  • npm test
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npx knip --include exports --reporter compact
  • npx slop-scan scan . --ignore "node_modules/**" --ignore "dist/**" --ignore "viewer/dist/**" --ignore "sideshow-term/dist/**" --json

This PR description was generated by Pi using GPT-5

@benvinegar benvinegar merged commit 61eab1f into main Jun 17, 2026
7 checks passed
@benvinegar benvinegar mentioned this pull request Jun 18, 2026
Merged
benvinegar added a commit that referenced this pull request Jun 18, 2026
@benvinegar @claude
docs(agents): add mermaid part kind, drop stale firstHtml ref (#59)
6e50afa 
The part-kind lists in AGENTS.md predated the mermaid part (#53) and
still referenced firstHtml, which no longer exists (#44). Sync the doc
to the actual SurfacePartKind: add mermaid; htmlPart is the only bridge.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@benvinegar