Skip to content

chore: stop publishing skills to npm - #1730

Merged
thymikee merged 3 commits into
mainfrom
agent/stop-publishing-skills
Aug 11, 2026
Merged

thymikee merged 3 commits into
mainfrom
agent/stop-publishing-skills

Conversation

@thymikee

@thymikee thymikee commented Aug 11, 2026 •

Copy link
Copy Markdown
Member

Summary

Stop shipping repository skills inside the npm CLI package. Skills remain discoverable and installable from GitHub, while npm now owns only the CLI/runtime distribution.

Keep npm installation as a user-owned prerequisite in the iOS Simulator and Android Emulator skills, while matching the canonical agent-device startup contract: begin normal work directly with platform-specific open, do not probe with version or help commands, and consult version-matched help only for specialized or unclear work. Apply the same open-first guidance to reusable agent rules and client examples in the setup documentation. Add packaging and startup regression guards, and align update wording with the new distribution boundary.

Touches 9 files; scope remains within skill distribution, packaging policy, and its documentation/tests.

Validation

pnpm check:affected --run passed every runnable local gate, including formatting, lint, typecheck, layering, Fallow, build, clean-install package verification, integration smoke, affected coverage, replay compatibility, and daemon wire compatibility.

The packed npm package contains no skills/** entries, and the focused packaging/skill/setup contract suite passes 21 tests. Runtime device validation does not apply because command behavior and platform implementations are unchanged.

@github-actions

github-actions Bot commented Aug 11, 2026 •

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 2.17 MB 2.17 MB +311 B
JS gzip 709.4 kB 709.4 kB +84 B
npm tarball 836.8 kB 835.6 kB -1.2 kB
npm unpacked 2.91 MB 2.91 MB -2.7 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 27.2 ms 27.4 ms +0.3 ms
CLI --help 66.1 ms 65.7 ms -0.5 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/session.js +190 B +58 B
dist/src/screenshot-result.js +61 B +32 B
dist/src/interaction.js +23 B +8 B
dist/src/selector-runtime.js +41 B +1 B

@github-actions

github-actions Bot commented Aug 11, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-11 09:47 UTC

@thymikee

Copy link
Copy Markdown
Member Author

Exact-head review at 5be3a2c: code/docs review is clean. Removing both skill paths from package.json’s explicit files allowlist excludes the repository skills from npm while preserving GitHub distribution; the real npm-pack/clean-consumer package lane and the manifest regression make that change load-bearing. The open-first skill/setup guidance matches the intended separately-installed CLI boundary, and its focused regressions fail against the old mandatory-probe wording. Runtime/device behavior is unchanged, so live device evidence is not applicable. All completed checks are green; Android and iOS Smoke Tests are still in progress. Ready for human review from a code/readiness standpoint, with final merge gated on those authoritative lanes.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Aug 11, 2026
@thymikee
thymikee merged commit b0d4b40 into main Aug 11, 2026
29 of 30 checks passed
@thymikee
thymikee deleted the agent/stop-publishing-skills branch August 11, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant