Skip to content

Point package metadata at figma/ and refresh dependencies - #41

Open
toeknee-figma wants to merge 1 commit into
mainfrom
chore/repo-metadata-and-deps
Open

Point package metadata at figma/ and refresh dependencies#41
toeknee-figma wants to merge 1 commit into
mainfrom
chore/repo-metadata-and-deps

Conversation

@toeknee-figma

Copy link
Copy Markdown
Collaborator

First of three stacked PRs sorting out the repo. This one is metadata and dependencies only — no behaviour changes — so it can merge on its own.

Metadata

repository, bugs and homepage still pointed at the Modyfi organisation, whose repository is now empty, so the npm package page led nowhere. Fixes #40.

This only reaches users once the package is republished. npm owners are currently tonyketcham and piers-modyfi, with no Figma org, so that step needs someone holding those credentials.

Dependencies

vite goes to 7.x as the version the plugin is typed and built against, and example/ gets its own pin at 5.4.21 so the integration build keeps exercising a version near the bottom of the >=3.2.7 range rather than the newest one.

Type resolution

CommonJS consumers were resolving to the ESM declarations (arethetypeswrong reported "Masquerading as ESM"). The export conditions now name index.d.cts for require and index.d.ts for import; all four resolution modes come back clean. files also listed a types directory that does not exist.

Verification

pnpm typecheck, pnpm build and pnpm example:build all pass.

🤖 Generated with Claude Code

The npm package still advertised the Modyfi organisation, whose repository
is now empty, so anyone following the package page landed nowhere (#40).

js-yaml moves to ^4.3.1, which carries the fix for GHSA-mh29-5h37-fv8m
(#38). Caret ranges replace exact pins on the runtime dependencies so
downstream installs can dedupe and pick up patches without a release here.

Type resolution for CommonJS consumers was falling back to the ESM
declarations; the export conditions now name index.d.cts for require and
index.d.ts for import.

Also drops c8, which no script invoked, and gives example/ its own vite
pin so the integration build exercises a version near the bottom of the
supported peerDependencies range rather than the one the plugin is built
against.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

npm package points to an empty repository Upgrade js-yaml Dependency

1 participant