Skip to content

chore(drift-sync): needs-human model-family decision (2026-08-14) - #376

Closed
copilotkit-bot[bot] wants to merge 1 commit into
mainfrom
drift-needs-human/2026-08-14-31777041713-811e6c6dc03dc3aa
Closed

chore(drift-sync): needs-human model-family decision (2026-08-14)#376
copilotkit-bot[bot] wants to merge 1 commit into
mainfrom
drift-needs-human/2026-08-14-31777041713-811e6c6dc03dc3aa

Conversation

@copilotkit-bot

Copy link
Copy Markdown

Needs a human decision (drift-sync)

The deterministic, zero-LLM drift-sync found a model-family change it must
NOT auto-apply (a genuinely new/unclassified family, or a registry
structural mismatch). It wrote the note file(s) below and opened this PR so
the decision is REACHABLE in the repo.

Not auto-merged — a human decides. For a new-family note: set the
note's decision line to Decision: include (to classify it) or delete the
note / close this PR (to reject), then merge this PR. The NEXT drift-sync
run reads the approved note from main and mechanically applies the registry
edit (still zero-LLM — a human-authored marker, not generated code), opening a
separate data-only PR. That two-run hand-off is how the loop closes.

Note file(s) in this PR

  • drift-proposals/gemini-gemini-3.7-flash-new-family.md

drift-sync outcome

npm warn Unknown project config "minimum-release-age". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
npm warn Unknown project config "block-exotic-subdeps". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
needs-human note(s) written this run — routed to human without a live re-collect (no registry edit to re-verify)
  [needs-human-new-family] gemini/gemini-3.7-flash: "gemini-3.7-flash" is unclassified — routed to human (drift-proposals/gemini-gemini-3.7-flash-new-family.md)
npm warn Unknown env config "block-exotic-subdeps". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
npm warn Unknown env config "minimum-release-age". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
npm warn Unknown project config "minimum-release-age". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
npm warn Unknown project config "block-exotic-subdeps". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
�[33m[STARTED]�[39m Backing up original state...
�[32m[COMPLETED]�[39m Backed up original state in git stash (ea47089)
�[33m[STARTED]�[39m Running tasks for staged files...
�[33m[STARTED]�[39m package.json�[0;90m — 1 file�[0m
�[33m[STARTED]�[39m *.{ts,mts,js,mjs,cjs,json,html,css,md}�[0;90m — 1 file�[0m
�[33m[STARTED]�[39m *.{ts,mts,js,mjs}�[0;90m — 0 files�[0m
�[33m[SKIPPED]�[39m *.{ts,mts,js,mjs}�[0;90m — no files�[0m
�[33m[STARTED]�[39m prettier --write
�[32m[COMPLETED]�[39m prettier --write
�[32m[COMPLETED]�[39m *.{ts,mts,js,mjs,cjs,json,html,css,md}�[0;90m — 1 file�[0m
�[32m[COMPLETED]�[39m package.json�[0;90m — 1 file�[0m
�[32m[COMPLETED]�[39m Running tasks for staged files...
�[33m[STARTED]�[39m Applying modifications from tasks...
�[32m[COMPLETED]�[39m Applying modifications from tasks...
�[33m[STARTED]�[39m Cleaning up temporary files...
�[32m[COMPLETED]�[39m Cleaning up temporary files...
[fix/drift-2026-08-14-31777041713 5b42042] fix(drift-sync): mechanical model-family sync (needs-human note file(s))
 1 file changed, 14 insertions(+)
 create mode 100644 drift-proposals/gemini-gemini-3.7-flash-new-family.md
unchecked-providers=
reason=needs-human
changeset-key=811e6c6dc03dc3aa

@pkg-pr-new

pkg-pr-new Bot commented Aug 14, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@copilotkit/aimock@376

commit: 5b42042

@jpr5

jpr5 commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Superseded by #378, which classifies both families: gemini-3.7-flash included, gemini-3.7-flash-video-understanding-eap excluded as gated pre-GA. Both notes are resolved and both checksum pins re-frozen there.

Note the procedure in this PR's body was wrong: Decision: include + merge reds its own gate, because adding to includeFamilies moves the checksum pin and drift-sync's allowlist bars it from re-pinning. #378 corrects that text and pins the correction with a test.

@jpr5 jpr5 closed this Aug 15, 2026
pull Bot pushed a commit to TheTechOddBug/aimock that referenced this pull request Aug 15, 2026
…xclude)

Google's live /models listing returned two families the drift canary could
classify as neither included nor excluded, firing needs-human on 2026-08-14
(run 31777041713, PR CopilotKit#376) and again on 2026-08-15 (run 31869038091, PR CopilotKit#377).

  - gemini-3.7-flash -> includeFamilies.gemini

    Stable GA text tier. Its live entry declares supportedGenerationMethods
    [generateContent, countTokens, createCachedContent, batchGenerateContent],
    byte-identical to the already-included gemini-3.6-flash / gemini-3.5-flash /
    gemini-2.5-flash / gemini-2.5-pro, with displayName "Gemini 3.7 Flash" and
    no preview/experimental/EAP marker. No bidiGenerateContent, no embedContent,
    no predict. Next release in the flash line aimock mocks at 3.5 and 3.6.

  - gemini-3.7-flash-video-understanding-eap -> excludeFamilies.gemini

    Gated pre-GA surface. The only one of the 54 live entries whose declared
    displayName AND description are "[Confidential] Gemini 3.7 Flash Video
    Understanding EAP". aimock does not mock pre-GA surfaces -- the same policy
    PREVIEW_FAMILY applies to every -preview tier -- but this id carries no
    trailing -preview token, so the rule cannot reach it and it is enumerated.

    Excluded on ACCESS STATUS, not on "video" appearing in the id. It really
    does declare generateContent (the same method set as its GA sibling), so it
    is text-capable; the disqualifier is provider-declared metadata, not a name
    substring.

Applied by hand WITH the re-pin, in one reviewed commit, because the automated
`Decision: include` two-run handoff cannot land this. drift-sync writes into
includeFamilies, drift-sync-check gate-2 re-runs logic-pin.test.ts, and gate-1's
changed-file allowlist admits only model-registry.ts and drift-proposals/ -- so
drift-sync cannot update the pin it just invalidated. Observed: the include add
alone moves the pin c2e2c56b -> 566fb89a and reds "freezes includeFamilies.gemini
membership"; the handoff would report gate-failed, not ok-applied. No commit in
this repo's history has ever carried drift-sync's "approved via drift-proposals"
marker -- every prior classification landed by hand the same way (72f85f8,
aa51d0c, 936b59c).

Both notes under drift-proposals/ are updated to RESOLVED with the decision and
its evidence, mirroring the openai gpt-transcribe / gpt-live-transcribe
precedent. Both families are added to the offline static /models wave in
models.drift.ts so test:drift covers them without a live key.

Verified against the live listing: zero unclassified gemini families remain.
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