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

chore: stop publishing skills to npm#1730
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