Repository navigation
build(deps): refresh Node types and transitive tooling - #100
Conversation
Bumps the development group with 1 update: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node). Updates `@types/node` from 26.6.3 to 26.6.4 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) --- updated-dependencies: - dependency-name: "@types/node" dependency-version: 26.6.4 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: development ... Signed-off-by: dependabot[bot] <support@github.com>
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs maintainer review before merge. Reviewed October 7, 2026, 4:41 AM ET / 08:41 UTC (Revision 7). ClawSweeper reviewWhat this changesRefreshes development dependencies and their lockfile resolutions, aligns Vitest ranges with the locked release, and records the update in the changelog. Merge readiness✅ Ready for maintainer review The dependency refresh remains useful and is not on current main. No actionable defect or unresolved merge concern was found; all 14 check runs succeeded on the exact head. Priority: P3 Review scores
Verification
How this fits togetherThe development toolchain compiles and tests libterminal’s shared terminal primitives for browser, Node.js, and Worker consumers. Dependency versions feed the build, type checking, coverage, and browser smoke checks. flowchart LR
A[Dependency ranges] --> C[Locked development tools]
B[Terminal source and tests] --> D[Build and validation]
C --> D
D --> E[Published JavaScript and types]
D --> F[Node and browser check results]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Keep the development toolchain current while preserving the runtime pin, supported Node versions, and dependency cooldown. Do we have a high-confidence way to reproduce the issue? Not applicable: this dependency-maintenance PR does not report a runtime bug; exact-head CI validates the refreshed toolchain. Is this the best way to solve the issue? Yes: the package ranges and lockfile are updated together without changing public runtime contracts or introducing a parallel implementation. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 7458d69ed8ed. LabelsLabel changes: No label changes. Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (6 earlier review cycles)
|
* chore(autoreview): use the shared canonical skill * docs(autoreview): preserve repository review priority
Refresh development dependencies after the two-day cooldown, preserving the Ghostty runtime pin and supported Node floors. Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
Merged as fc0f565 after all checks passed on exact head 94b171a. The tested candidate and squash commit have identical Git trees. Validation on a clean Linux machine: PR CI and post-merge main CI passed, including Node 22/24 on Linux, macOS, and Windows, Node 26 on Linux, browser smoke, and workflow lint. Post-merge CodeQL also passed. No CI reruns were needed. Thanks @dependabot[bot]. The Ghostty runtime, package version, runtime floors, and cooldown remain unchanged; no release was performed. |
Refresh development dependencies after the repository's two-day cooldown: Node.js types 26.6.4, Rolldown 1.2.12 and its platform bindings, Oxc types, and compatible test/build transitive patches. Align the Vitest and V8 coverage ranges with the already-locked 5.0.3 patch release.
The Ghostty runtime remains pinned at 0.4.0. The package version, supported Node.js floors, pnpm 11.28.0, and dependency cooldown are unchanged. Includes an Unreleased entry crediting @dependabot[bot].
Validation passed:
pnpm check(140 unit tests),pnpm test:coverage(84.84% statements), andpnpm test:browser(9 tests).Verified head:
94b171a974c7287c6a1b2572430bbbd56bd5f3f2. CI and CodeQL passed.