chore(deps-dev): bump the dev-dependencies group across 1 directory with 4 updates - #154
Merged
salishforge merged 1 commit intoJul 25, 2026
Conversation
…ith 4 updates --- updated-dependencies: - dependency-name: "@types/node" dependency-version: 26.1.1 dependency-type: direct:development update-type: version-update:semver-major dependency-group: dev-dependencies - dependency-name: eslint dependency-version: 10.7.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-dependencies - dependency-name: tsx dependency-version: 4.23.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-dependencies - dependency-name: typescript dependency-version: 7.0.2 dependency-type: direct:development update-type: version-update:semver-major dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
deleted the
dependabot/npm_and_yarn/dev-dependencies-42e5b04980
branch
July 25, 2026 22:34
This was referenced Jul 26, 2026
salishforge
added a commit
that referenced
this pull request
Jul 26, 2026
…ain (#167) The dev-dependencies bump to typescript@^7.0.2 (#154) broke `npm ci` repo-wide: every @typescript-eslint release through 8.65.0 declares peer typescript '>=4.8.4 <6.1.0', so dependency resolution fails with ERESOLVE before any job can run. Main and every open PR have been red since. Dependabot's own follow-up (#158, eslint-plugin 8.65.0) cannot fix it — 8.65.0 carries the same <6.1.0 cap. ~6.0.3 is the newest TypeScript the toolchain supports (and the exact version the code has been verified against). Revisit when typescript-eslint publishes TS 7 support. Verified: npm ci, tsc --noEmit, eslint, tsc build all pass locally. Claude-Session: https://claude.ai/code/session_011xCqQo49d3CEbn6oEvb3Ru Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
salishforge
added a commit
that referenced
this pull request
Jul 27, 2026
* fix(build): declare the optional @huggingface/transformers module `npm run build` is a plain `tsc`, but embedding.ts's only use of this package is a dynamic import of an *optional* dependency. Environments that install without optional deps — CI does so intermittently, since the package pulls platform-specific native artifacts — fail resolution and die with TS2307 on an unchanged source tree. That made Build a coin-flip rather than a signal (it is what failed on #168). The declaration is minimal by design: one factory, returning `unknown`, because embedding.ts already defines its own FeatureExtractionPipeline shape and casts to it. No upstream type is depended on, so shadowing them when the package is present costs nothing. Verified all three states: package present (passes), package absent (passes), and package absent without this declaration (reproduces TS2307 at embedding.ts:208). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011xCqQo49d3CEbn6oEvb3Ru * ci(deps): stop dependabot proposing typescript past the lint peer ceiling Every published @typescript-eslint release declares peer typescript ">=4.8.4 <6.1.0". A bump past that ceiling fails `npm ci` with ERESOLVE before any job starts, so it reds out every open PR in the repo rather than only its own — which is what happened for a day after #154 landed typescript@^7.0.2, and is what open PR #169 is currently retrying. Expressed as a version ceiling rather than `update-types: version-update:semver-major`, because 6.1.0 is a *minor* bump that would breach the same peer range and slip through a major-only ignore. CI already fails fast and loudly on this, so the rule is about not burning CI minutes and reviewer attention on PRs that cannot pass — not a substitute for the gate. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011xCqQo49d3CEbn6oEvb3Ru --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.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.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions