Skip to content

docs(changelog): generateDiff fallback wording; scope smartapply failure to the model path - #12

Draft
mikkel wants to merge 1 commit into
mainfrom
docs/changelog-ember-fixes
Draft

mikkel wants to merge 1 commit into
mainfrom
docs/changelog-ember-fixes

Conversation

@mikkel

@mikkel mikkel commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Follow-up to #11, with Ember's two changelog fixes. The only file changed is CHANGELOG.md, under [0.3.1] — Unreleased.

  1. The summary no longer says "No change to generateDiff." fix: replace retired default model with deepseek/deepseek-v4.1-flash-1017 #11 did change it: with no model and no GPTDIFF_MODEL, generateDiff now falls back to a working DEFAULT_MODEL (deepseek/deepseek-v4.1-flash-1017) instead of the retired id, which returned HTTP 410. "What smartapply writes is unchanged" stays.
  2. The smartapply failure is now scoped to the model-backed path. smartapply first applies each patch deterministically from the +/- lines (applyPatchToFile, in the fast-path branch of smartapply in src/gptdiff.js), and it handles deletions without a model call. Only files that fall through to callLlmForApplyWithThink hit the retired default, and the entry now says so.

Companion: 255BITS/patchling-py#25 makes fix 2 in its changelog.

No code changes, version bump, tag, or publish. Local npm test: 248 tests, 244 passed, 0 failed, 4 skipped (live), the same as main.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying patchling with  Cloudflare Pages  Cloudflare Pages

Latest commit: a4d0b7a
Status: ✅  Deploy successful!
Preview URL: https://8e78d5bf.patchling.pages.dev
Branch Preview URL: https://docs-changelog-ember-fixes.patchling.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant