Package the skill analyzer with standalone artifact handling - #42
Open
expo-tuft[bot] wants to merge 7 commits into
Open
expo-tuft[bot] wants to merge 7 commits into
expo-tuft[bot] wants to merge 7 commits into
Conversation
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
August 13, 2026 08:37
7141d7a to
b047994
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
August 13, 2026 08:56
b047994 to
f517c43
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
August 15, 2026 15:38
f517c43 to
f010b89
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 11:14
f010b89 to
76d43c3
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 11:29
76d43c3 to
d630ce2
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 11:50
d630ce2 to
f10fb7d
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 12:34
3558a6d to
250dc09
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
2 times, most recently
from
September 10, 2026 12:38
0851905 to
aecd40c
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 12:46
aecd40c to
405d855
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 18:35
4de2ac3 to
cfab8a3
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 18:44
cfab8a3 to
7418dd2
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 18:49
7418dd2 to
5e5bd8a
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 18:58
5e5bd8a to
ea862fa
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 19:15
ea862fa to
9357dce
Compare
expo-tuft
Bot
force-pushed
the
@kudo/skill-analyzer-package
branch
from
September 10, 2026 19:55
9357dce to
e32eec6
Compare
Kudo
requested review from
adi-georgian and
kartik-georgian
and removed request for
adi-georgian
September 11, 2026 19:13
Kudo
approved these changes
Sep 11, 2026
…oints Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
…rypoint Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
Co-authored-by: kudo <46429+kudo@users.noreply.github.com>
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
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.
Extracts skill-use analysis into
@expo/skill-analyzer, a Bun package with compiled ESM, bundled private source-scan code/types, packaged check data, and an installedskill-analyzerexecutable. Babel and tar remain external runtime dependencies. Requires Bun >=1.3.14.The package is the only analyzer implementation and entrypoint. Removes old repository import/CLI shims and check-data symlinks; harness tests, reporting imports, authoring, and EAS workflows use the package directly.
analyze-artifactsrequires explicit--prd-skills;materializeandbundle-checkprovide the utility commands formerly invoked through repository files. Workflow jobs install workspace dependencies before using compiled packages. The repository's shell orchestration selects its own dataset explicitly.Scoring, report schemas, and safe artifact extraction remain covered by regression tests and fixed pre-extraction goldens. Published modules share registry/class identity; declarations require neither Bun ambient types nor TypeScript source imports. Releases use
bun pm packto resolve catalog/workspace versions.Migration guide for the follow-up expo/skills PR covers installation, the two CI analysis call sites, explicit ground truth, report paths, utility commands/imports, and baseline cache invalidation. No expo/skills changes or npm publication are performed here.
Validation: typecheck and 315 TypeScript tests at this layer; 33 targeted Python artifact/workflow tests; npm and Bun isolated analyzer tarball checks (CLI/API, all exports, private registry blocked, materialization, directory/archive inputs, fixed golden parity, and strict NodeNext/Bundler declarations); formatting, shell syntax, and local YAML parsing pass. The downstream release helper's 46 tests also pass after restacking.
EAS server-side validation was attempted for all four changed workflows but the current account cannot read the linked Expo project (
Entity not authorized). Local YAML parsing is not a substitute for that validation; it remains to be run by an account with project access. No live workflow/model runs were triggered.Review / merge order: #40 → #47 → #51 → #55 → #41 → #57 → #59 → #53 → #42 → #50. Stacked on #53.
Clean-install correction: the checked-in
bin/skill-analyzer.mjsloads the compiled CLI and exists before Bun links workspace executables. Updated Bun’s cached workspace bin path in the lockfile. Verified a fresh checkout with frozen installation, the installed CLI, and all 315 TypeScript tests.Test Plan
manual eval run https://expo.dev/accounts/expo-ci/projects/expo-skills/workflows/01a091b6-82d3-7372-ae39-e7967775911a