Skip to content

Align monaco-code-editor with Cosmos repo conventions and switch library build to tsup#30

Open
anil08607 wants to merge 1 commit into
tscircuit:mainfrom
anil08607:build-site
Open

Align monaco-code-editor with Cosmos repo conventions and switch library build to tsup#30
anil08607 wants to merge 1 commit into
tscircuit:mainfrom
anil08607:build-site

Conversation

@anil08607

Copy link
Copy Markdown

What Changed

  • Replaced the old build step, which only typechecked, with a real tsup library build
  • Published compiled artifacts from dist/ instead of exporting raw src/ files
  • Added a shared tsup.config.ts with browser-targeted ESM output, declarations, sourcemaps, and CSS inclusion
  • Kept Cosmos as the local preview/site workflow by adding matching scripts like start, build:site, cosmos-export, and vercel-build
  • Updated package exports so consumers resolve dist/index.js, dist/index.d.ts, and the generated stylesheet

Why We Changed This

  • The repo was using Cosmos for preview, but build was not producing a distributable package
  • Publishing src/index.ts directly is inconsistent with sibling repos and can break downstream consumers expecting built JS and type declarations
  • pcb-viewer and similar repos already separate concerns cleanly:
    • cosmos and cosmos-export for demos and site checks
    • tsup for npm package output
  • Matching that convention makes this package easier to publish, preview, and maintain across the repo ecosystem

@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
monaco-code-editor Ready Ready Preview, Comment Jul 12, 2026 3:31am

Request Review

@rushabhcodes rushabhcodes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anil08607 use order-dialog for ref, this is a componemt lib, it exports src directly, dont change that. afk rn take review from @imrishabh18 @mohan-bee

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