More cleanup: unused devDependency, undiscovered examples, superfluous exports - #1525
Merged
Conversation
Zero imports anywhere in ai-autopilot/src; the only "playwright" hit in the package is a filename-pattern regex (config file detection), not a use of the npm package.
autopilot-quickstart, bootstrap-quickstart, and framework-demo all built and typechecked fine, but nothing in the repo links to them — no root README, no package README, no doc references any of the three by path. They were only findable by browsing examples/, so they weren't paying for their token overhead.
38 functions/consts across the two packages were exported but never imported anywhere else in the repo, and none are re-exported through a public subpath barrel (ai-sdk's ./node, ./gateway, ./eval, ./computer-use, ./react; the-framework's ./dashboard-rpc, ./client) — so `export` was just noise. Verified each one repo-wide (including package.json exports maps and wildcard re-exports) before stripping the keyword; declarations are otherwise untouched.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Three independent, separately-committed cleanups from the ongoing token-burn reduction (follows #1522, #1523, #1524):
playwright-core— unused devDependency inpackages/ai-autopilot. Zero imports; the only "playwright" hit in the package is an unrelated filename-pattern regex.examples/autopilot-quickstart,examples/bootstrap-quickstart,examples/framework-demo— all three built and typechecked fine, but nothing in the repo links to them (no README, no doc references). Only findable by browsingexamples/, so they weren't paying for their token overhead.packages/ai-sdk/srcandpackages/the-framework/src— functions/consts exported but never imported anywhere else in the repo, and not re-exported through any published subpath barrel (checkedpackage.jsonexportsmaps and wildcard re-exports for both packages before touching anything). Only theexportkeyword was stripped; declarations are otherwise untouched.Verification
pnpm install— lockfile regenerated cleanlypnpm typecheck— all 8 tasks passpnpm build— all packages build, dashboard bundles intothe-frameworkpnpm test— 7 test tasks pass, including framework-dashboard's 81 files / 797 testsTest plan
pnpm installpnpm typecheckpnpm buildpnpm testexportkeywordGenerated by Claude Code