[PR #201] [MERGED] fix(parity): refresh the Python snapshot and retire the discovery-methods lead #201

New issue

Closed

opened 2026-10-31 22:05:38 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 22:05:38 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-typescript-sdk/pull/201
Author: @mattmillerai
Created: 10/9/2026
Status: ✅ Merged
Merged: 10/9/2026
Merged by: @mattmillerai

Base: main ← Head: matt/parity-refresh-discovery-lead

📝 Commits (2)

6fc8714 fix(parity): refresh the Python snapshot and retire the discovery-methods lead
bf6cb9e docs(agents): state that neither lead field carries an entry today

📊 Changes

3 files changed (+6 additions, -14 deletions)

View changed files

📝 AGENTS.md (+2 -2)
📝 parity/python-surface.json (+4 -0)
📝 src/sdk/surface-parity.test.ts (+0 -12)

📄 Description

Summary

sdk-parity is red on main and on every open PR. #167 fixed the extractor, but the Python SDK has moved since that PR's snapshot was taken: it now ships models.list and models.schema. That fires the rot guard on the discovery-methods-land-first-in-typescript lead.

Refresh parity/python-surface.json with pnpm sync:python-surface.
Delete the now-stale lead entry, so list/schema are compared directly again.
Update AGENTS.md: seven asymmetries are declared now, and no models method is leading.

ELI5

The test that checks both SDKs expose the same methods was still saying "TypeScript has list and schema first, Python will catch up." Python has caught up, so that note is removed and the snapshot of Python's methods is updated.

Provenance

Found while shepherding the open PR set: sdk-parity failed on all of them after #167 landed. Verified locally on top of current main: lint, format, typecheck, pnpm test, and node scripts/sync-python-surface.mjs (OK against comfy-python-sdk@main).

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-typescript-sdk/pull/201 **Author:** [@mattmillerai](https://github.com/mattmillerai) **Created:** 10/9/2026 **Status:** ✅ Merged **Merged:** 10/9/2026 **Merged by:** [@mattmillerai](https://github.com/mattmillerai) **Base:** `main` ← **Head:** `matt/parity-refresh-discovery-lead` --- ### 📝 Commits (2) - [`6fc8714`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/6fc87144ed618981f3415ca638f08993d6fe4d43) fix(parity): refresh the Python snapshot and retire the discovery-methods lead - [`bf6cb9e`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/bf6cb9e8844ed9b81afbcc9b5704ab26e37e8697) docs(agents): state that neither lead field carries an entry today ### 📊 Changes **3 files changed** (+6 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `AGENTS.md` (+2 -2) 📝 `parity/python-surface.json` (+4 -0) 📝 `src/sdk/surface-parity.test.ts` (+0 -12) </details> ### 📄 Description ## Summary `sdk-parity` is red on `main` and on every open PR. #167 fixed the extractor, but the Python SDK has moved since that PR's snapshot was taken: it now ships `models.list` and `models.schema`. That fires the rot guard on the `discovery-methods-land-first-in-typescript` lead. - Refresh `parity/python-surface.json` with `pnpm sync:python-surface`. - Delete the now-stale lead entry, so `list`/`schema` are compared directly again. - Update AGENTS.md: seven asymmetries are declared now, and no `models` method is leading. ## ELI5 The test that checks both SDKs expose the same methods was still saying "TypeScript has `list` and `schema` first, Python will catch up." Python has caught up, so that note is removed and the snapshot of Python's methods is updated. ## Provenance Found while shepherding the open PR set: `sdk-parity` failed on all of them after #167 landed. Verified locally on top of current `main`: lint, format, typecheck, `pnpm test`, and `node scripts/sync-python-surface.mjs` (OK against `comfy-python-sdk@main`). 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 22:05:38 +00:00