Skip to content

Correct two stale statements in the vendor patch ledgers #1039

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
TheGreatAxios merged 1 commit into from
Sep 14, 2026
+13 −9
Merged

Correct two stale statements in the vendor patch ledgers #1039

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions docs/VENDORING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -227,14 +227,14 @@ those packages now resolves to the single root instance. As of this sync,

## Patched vs. verbatim

Of the eleven vendored paths, four carry local patches:
Of the twelve vendored paths, four carry local patches:
`@intx/inference`, `@intx/types`, `@intx/storage-isogit`, and
`@intx/agent` — every divergence is a real fix not yet upstream, listed
one-per-patch in that package's `PATCHES.md`. The other seven paths
(six whole packages plus the partial `vendor/intx-workflow-host/adapters/`
tree) are verbatim copies of upstream — no modifications. A diff against a
one-per-patch in that package's `PATCHES.md`. The other eight paths
(six whole packages plus the two partial `vendor/intx-workflow-host/`
paths) are verbatim copies of upstream — no modifications. A diff against a
pristine upstream checkout at the same commit and paths will show 100%
upstream-authored lines for those seven.
upstream-authored lines for those eight.

## Notable upstream shape changes carried by the 2026-08-22 sync

Expand Down
12 changes: 8 additions & 4 deletions vendor/intx-inference/PATCHES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -542,12 +542,16 @@ requires instead of the `thinking:{type:"enabled",budget_tokens}` shape
adaptive-only models reject. Upstream's list lacks the model. Consumed by the
`first-class-providers` registry (`claude-fable-5-1` is a shipped, selectable
anthropic and zen model; CHANGELOG 0.3.17 advertises adaptive thinking for
Fable 5) and guarded by the "adaptive thinking request shape" suite in
`providers/anthropic.test.ts`.
Fable 5). The companion "adaptive thinking request shape" suite that once
guarded the wire shape in `providers/anthropic.test.ts` was dropped by the
2026-09-14 pristine overwrite (`5a291aad5`) and never re-applied, so the
wire shape currently rides without an in-tree guard; only the model id is
covered, by the first-party `first-class-providers` registry test.

**Disposition:** Re-carryable — a one-line list addition that survives sync
trivially; the guard suite re-applies verbatim. Risk: upstream may grow its
own adaptive-models list; reconcile the two on next sync. **Removal path:**
trivially. Risk: upstream may grow its own adaptive-models list; reconcile
the two on next sync, and consider restoring the dropped guard suite.
**Removal path:**
Upstream adding `claude-fable-5-1` to its own `ADAPTIVE_THINKING_MODELS`.

---
Expand Down
Loading