fix(autolink): detect create extension npm bin symlinks

[jianliang00]

Summary

Fix create-lynx-extension so the published CLI runs when invoked through npm-created bin symlinks such as npm create lynx-extension-canary or npm exec create-lynx-extension.

Root Cause

The executable guard compared path.resolve(process.argv[1]) directly with fileURLToPath(import.meta.url). npm invokes package bins through node_modules/.bin/* symlinks, so the comparison failed and main() was never called. The process exited successfully without printing help, prompting, or creating files.

Changes

• Normalize the CLI entrypoint and module path through fs.realpathSync before comparing them.
• Export the entrypoint helper for focused test coverage.
• Add a regression test for npm bin symlink execution.

Validation

• pnpm --filter create-lynx-extension test
• pnpm --filter create-lynx-extension build
• pnpm biome check packages/lynx/create-lynx-extension/src/cli.ts packages/lynx/create-lynx-extension/test/cli.test.ts
• pnpm dprint fmt packages/lynx/create-lynx-extension/src/cli.ts packages/lynx/create-lynx-extension/test/cli.test.ts
• Local npm exec --package /Users/bytedance/Code/lynx-stack/packages/lynx/create-lynx-extension create-lynx-extension -- --help

Summary by CodeRabbit

• Bug Fixes

  • Improved CLI entrypoint detection to correctly handle npm .bin symlinks and consistently normalize paths for reliable executable detection.

• Tests

  • Added test coverage to verify npm .bin symlink entrypoint detection for the CLI.

[changeset-bot]

🦋 Changeset detected

Latest commit: b3eaa58

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
create-lynx-extension	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR refactors CLI entrypoint detection: isCliEntrypoint now accepts optional entrypoint and moduleUrl, normalizes and realpath-resolves both paths, and compares them. Tests mock fs.realpathSync to verify npm .bin symlink detection; a changeset documents the fix.

Changes

CLI Entrypoint Symlink Detection

Layer / File(s)	Summary
Entrypoint detection with real-path normalization packages/lynx/create-lynx-extension/src/cli.ts	Added fs import and refactored isCliEntrypoint(entrypoint = process.argv[1], moduleUrl = import.meta.url) to normalize absolute paths, use fs.realpathSync with fallback, and compare resolved paths.
Symlink entrypoint test coverage packages/lynx/create-lynx-extension/test/cli.test.ts	Extended imports to include pathToFileURL and isCliEntrypoint. Added a test that mocks fs.realpathSync so .bin/create-lynx-extension resolves to dist/cli.js and asserts detection returns true.
Release changeset .changeset/fix-create-extension-bin-entrypoint.md	Adds a patch changeset noting a fix for npm bin symlink entrypoint detection for create-lynx-extension.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

• PupilTong
• f0rdream
• colinaaa

Poem

🐰 I hopped along a symlink trail,
Real-path made the puzzle pale.
Entrypoint matched with tidy grace,
.bin and dist now share the place.
Cheers — a bunny's little celebratory dance! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	⚠️ Warning	The title mentions fixing autolink but the PR actually fixes create-lynx-extension bin symlink detection, which is unrelated to autolink.	Change the title to reflect the actual change, such as 'fix(create-lynx-extension): detect npm bin symlinks' or 'fix: detect create-lynx-extension npm bin symlinks'.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch p/wangjianliang/fix-create-lynx-extension-bin-entrypoint

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
4582	1	4581	63

View the top 1 failed test(s) by shortest run time

tests/web-elements.spec.ts::web-elements test suite › x-refresh-view › x-refresh-view/pull

Stack Traces | 3.51s run time

web-elements.spec.ts:3179:5 x-refresh-view/pull

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[Copilot]

Pull request overview

This PR fixes create-lynx-extension’s CLI executable guard so main() runs correctly when the CLI is invoked via npm-created node_modules/.bin/* symlinks (e.g. npm create lynx-extension-canary, npm exec create-lynx-extension), instead of silently exiting.

Changes:

• Normalize both the invoked entrypoint path and the module path using fs.realpathSync before comparing them in the CLI “am I the entrypoint?” guard.
• Export the isCliEntrypoint helper to enable direct regression testing.
• Add a regression test that simulates npm bin symlink resolution via a fs.realpathSync mock.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/lynx/create-lynx-extension/src/cli.ts	Realpath-normalize the entrypoint/module paths for correct CLI guard behavior when invoked through symlinks; export isCliEntrypoint.
packages/lynx/create-lynx-extension/test/cli.test.ts	Add regression coverage for npm .bin symlink execution by exercising isCliEntrypoint with mocked realpathSync.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codspeed-hq]

Merging this PR will degrade performance by 15.39%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 2 regressed benchmarks
✅ 79 untouched benchmarks
⏩ 26 skipped benchmarks¹

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
❌	008-many-use-state-destroyBackground	8 ms	9.5 ms	-16.16%
❌	transform 1000 view elements	40 ms	46.9 ms	-14.62%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

---

_{Comparing p/wangjianliang/fix-create-lynx-extension-bin-entrypoint (b3eaa58) with main (9437b0d)}

Footnotes

1. 26 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[relativeci]

React External

#1275 Bundle Size — 693.04KiB (0%).

b3eaa58(current) vs 9437b0d main#1274(baseline)

Bundle metrics  no changes

	Current #1275	Baseline #1274
Initial JS	0B	0B
Initial CSS	0B	0B
Cache Invalidation	0%	0%
Chunks	0	0
Assets	3	3
Modules	17	17
Duplicate Modules	5	5
Duplicate Code	8.59%	8.59%
Packages	0	0
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #1275	Baseline #1274
Other	693.04KiB	693.04KiB

Bundle analysis report Branch p/wangjianliang/fix-create-lynx-... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[relativeci]

React Example with Element Template

#427 Bundle Size — 197.79KiB (0%).

b3eaa58(current) vs 9437b0d main#426(baseline)

Bundle metrics   2 changes

	Current #427	Baseline #426
Initial JS	0B	0B
Initial CSS	0B	0B
Cache Invalidation	0%	0%
Chunks	0	0
Assets	4	4
Modules	79(-2.47%)	81
Duplicate Modules	23	23
Duplicate Code	40.32%(+0.07%)	40.29%
Packages	2	2
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #427	Baseline #426
IMG	145.76KiB	145.76KiB
Other	52.03KiB	52.03KiB

Bundle analysis report Branch p/wangjianliang/fix-create-lynx-... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[relativeci]

React MTF Example

#1293 Bundle Size — 207.46KiB (0%).

b3eaa58(current) vs 9437b0d main#1292(baseline)

Bundle metrics  no changes

	Current #1293	Baseline #1292
Initial JS	0B	0B
Initial CSS	0B	0B
Cache Invalidation	0%	0%
Chunks	0	0
Assets	3	3
Modules	192	192
Duplicate Modules	77	77
Duplicate Code	44.38%	44.38%
Packages	2	2
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #1293	Baseline #1292
IMG	111.23KiB	111.23KiB
Other	96.23KiB	96.23KiB

Bundle analysis report Branch p/wangjianliang/fix-create-lynx-... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[relativeci]

Web Explorer

#9735 Bundle Size — 901.38KiB (0%).

b3eaa58(current) vs 9437b0d main#9734(baseline)

Bundle metrics  no changes

	Current #9735	Baseline #9734
Initial JS	45.06KiB	45.06KiB
Initial CSS	2.22KiB	2.22KiB
Cache Invalidation	0%	0%
Chunks	9	9
Assets	11	11
Modules	229	229
Duplicate Modules	11	11
Duplicate Code	27.22%	27.22%
Packages	10	10
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #9735	Baseline #9734
JS	497.1KiB	497.1KiB
Other	402.06KiB	402.06KiB
CSS	2.22KiB	2.22KiB

Bundle analysis report Branch p/wangjianliang/fix-create-lynx-... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[relativeci]

React Example

#8160 Bundle Size — 236.51KiB (0%).

b3eaa58(current) vs 9437b0d main#8159(baseline)

Bundle metrics  no changes

	Current #8160	Baseline #8159
Initial JS	0B	0B
Initial CSS	0B	0B
Cache Invalidation	0%	0%
Chunks	0	0
Assets	4	4
Modules	197	197
Duplicate Modules	80	80
Duplicate Code	44.87%	44.87%
Packages	2	2
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #8160	Baseline #8159
IMG	145.76KiB	145.76KiB
Other	90.75KiB	90.75KiB

Bundle analysis report Branch p/wangjianliang/fix-create-lynx-... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}