Repository navigation
genclass 0.3.4: GitHub App for every GitHub repository, hand-written and integration setups, commands that run as printed - #222
Conversation
…ten and integration setups recognised, and commands that run as printed From a review of every setup path: the GitHub App step waited for Playwright tests; a hand-written setup looped at 'Put this app in your account'; @genclass/nuxt and @genclass/astro apps were treated as new; connect --yes could replace a teammate's key; setup --yes --json mixed the view into the JSON; status --json pointed agents at a command that does nothing without a terminal; and several printed commands did not run as typed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
GenClass runtime review2 findings in this pull request's tests at
Reply The race report, test by testRace report2 signatures in 2 tests, 2 reproducible (flagged in 2 or more seeds of the same test) · 19 files read · mode observe
[chromium] search.spec.ts > search > buggy page: results of the last query2 seeds (1, 2) · 1 signature · chaos: latency 0–600 ms, reorder Witnesses (facts measured without the model)
search.spec.ts > search > buggy page: results of the last query5 seeds (1, 2, 3, 4, 5) · 1 signature · chaos: latency 0–600 ms, reorder Witnesses (facts measured without the model)
3 tests with nothing flagged
Lines under "Witnesses (facts measured without the model)" are happens-before facts the runtime measured in the page without the model, one per signature (witness kind, operation): overwrote-newer, a response wrote a field over what a newer request or the user had written since the request started; identical-in-flight, two identical requests in flight together from separate user actions. They carry no confidence: a fact is listed when it happened. Reproducible: seen in 2 or more seeds of the same test. Whether a fact is a bug is for you to decide (some overwrites are intended; many are repaired a moment later). genclass-dev[bot] read 19 race files from the artifacts |
genclass0.3.4: fixes from a review of every setup path after a teammate hit the 0.3.2 bug. Each one was proven by running it, except where noted.init.mjs inspectProject).@genclass/nuxt/@genclass/astroapps were treated as new; the walk would add a second runtime import and a second app.connect --yesno longer replaces a key that is none of the account's apps (a teammate's) unless--appor--namesays which. A key from the person's own.genclass.localis still taken.setup --yes --jsonprints only JSON;status --jsonnamesnpx genclass setup --yesinstead of a command that does nothing without a terminal.dashboard --all/--range/--app;connect,init,auditat the home prompt (were sent to the model as questions); the CSP advice'sfetch-model;scanof a local address; long or quoted names after--app; hints that dropped what was typed.removefrom a folder inside the app;init/connectthrough genclass add no overlay, name themselvesnpx genclass, and point a newer runtime atnpx genclass@latest.Not changed (noted by the review, left for later): Vite 7 settings for frameworks other than plain Vite (needs a real build to confirm), and
setup --yesstill waits for the sign-in as the README documents.Tests: CLI 181 (3 new), the runtime's install tests 157,
tscclean. Smoke-tested from the built file: signed-outdashboard --all, theconnecterror namingnpx genclass,scanof localhost,status --json.🤖 Generated with Claude Code