Skip to content

chore: 1.20.1 - #3388

Merged
janicduplessis merged 1 commit into
mainfrom
chore/release-1.20.1
Oct 10, 2026
Merged

janicduplessis merged 1 commit into
mainfrom
chore/release-1.20.1

Conversation

@janicduplessis

@janicduplessis janicduplessis commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Description

The maintainer approved npm 1.20.1 on 2026-10-10. It ships #3382, which replaces the xcode package with an in-tree project.pbxproj parser so npm install stim and npx stim stop printing the uuid@7.0.3 deprecation warning. npm has 1.20.0 on latest.

Solution

Pre-tag QA is omitted, as with 1.18.0 to 1.20.0; the notes' QA section says so and records the one native check that did run.

Test plan

  • Native Xcode acceptance (Native E2E, native_ios_only, fixture) on main at eafc8cc, which contains the new parser: run 38083829755 passed.
  • Local on this commit: format:check, lint, build, typecheck, knip, release:prep --check, CLI --version (1.20.1), --help, guide agent, guide tutorial manual (prints Unknown section "manual"), licenses:check, and pnpm tarball inspection of the seven packages (README and LICENSE present, versions 1.20.1, no workspace: ranges, no xcode dependency in stim).
  • Unit, e2e, runtime and the full Windows suite run in CI on the exact main commit that gets tagged.

@janicduplessis janicduplessis added area: cli packages/stim-cli, including its README and guide area: mobile apps/mobile and apps/demo-server area: docs docs/ and root documents labels Oct 10, 2026
@janicduplessis

Copy link
Copy Markdown
Collaborator Author

Fresh review of ee95737 (not the implementer): clear.

  • Versions: all seven package.json files go 1.20.0 to 1.20.1 and nothing else in them changes. apps/mobile licenses.json changes only the @stim-cli/core version.
  • docs/releases/1.20.1.md checked against git diff v1.20.0..origin/main -- packages:
    • Both commit SHAs exist and are linked in full.
    • e8f1499: xcode is gone from stim's dependencies, @xmldom/xmldom stays, and the in-tree parser is in the diff.
    • 52eba84: the run section gains the agent-device check with the installed-or-not fallback, the parallel prompt is added, and the manual section is deleted.
    • manual is no longer linked anywhere on origin/main website. v1.20.0 linked it from desktop.md:137 and getting-started.md:106.
    • Nothing is overclaimed, and the QA section states its deviation from RELEASE.md section 3.
  • Run 38083829755: success, headSha eafc8cc, which includes the parser commit. The native Xcode acceptance job passed and the other jobs were skipped, as the notes say.
  • needs-human: not required. This PR is a version bump and notes only, and touches no area in the AGENTS.md list.

Non-blocking nit: b52e08b changed the ios.scheme description in packages/core/state/settings-registry.ts to add "the --scheme flag overrides it". That text appears in stim settings and the shipped schema, and the notes do not mention it. It is a one-line wording change, so I did not block on it.

CI for the exact commit still has to be green before merge.

@janicduplessis janicduplessis added the review: ai-approved A fresh AI review found nothing left to fix label Oct 10, 2026
@janicduplessis
janicduplessis marked this pull request as ready for review October 10, 2026 20:48
@janicduplessis janicduplessis removed the review: ai-approved A fresh AI review found nothing left to fix label Oct 10, 2026
@janicduplessis

Copy link
Copy Markdown
Collaborator Author

Re-review of f772cbd: one actionable finding, so no review: ai-approved label yet.

docs/releases/1.20.1.md line 18 still says "49 changed paths, 5 mapped to a row". node scripts/release-qa-matrix.mjs v1.20.0 run in .worktrees/release-1.20.1 reports 76 changed paths, 7 mapped, 0 unclassified. Please update the numbers and push.

Everything else checks out:

A push after this review will need a new fresh review.

@janicduplessis

Copy link
Copy Markdown
Collaborator Author

Fresh review of 067c4e6 (not the implementer): clear.

  • Seven package.json files change only 1.20.0 to 1.20.1; apps/mobile licenses.json changes only the @stim-cli/core version.
  • docs/releases/1.20.1.md covers every user-facing package change since v1.20.0 (e8f1499, 52eba84, b52e08b). The other commits in the range (b2b5230, desktop polish) touch tests or apps only. Full SHAs resolve. The xcode dependency claim checks out: absent in v1.19.0, present in v1.20.0, gone on main, with @xmldom/xmldom kept.
  • Matrix counts are fixed: node scripts/release-qa-matrix.mjs v1.20.0 reports 76 changed paths, 7 mapped, 0 unclassified, 8 of 8 rows. Run 38083829755 is Native E2E on eafc8cc (an ancestor of main), conclusion success, native Xcode acceptance job passed. The QA section states the omitted pre-tag QA and the RELEASE.md section 3 deviation.
  • Nothing requires review: needs-human.

@janicduplessis janicduplessis added the review: ai-approved A fresh AI review found nothing left to fix label Oct 10, 2026
@janicduplessis
janicduplessis merged commit a196b6e into main Oct 10, 2026
30 checks passed
@janicduplessis
janicduplessis deleted the chore/release-1.20.1 branch October 10, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: cli packages/stim-cli, including its README and guide area: docs docs/ and root documents area: mobile apps/mobile and apps/demo-server review: ai-approved A fresh AI review found nothing left to fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant