Skip to content

Fix Etsy listing includes, Telegram getUpdates timeout, Odoo message_post hint - #1324

Merged
helpcodeai merged 2 commits into
mainfrom
fix/adapter-params-1029
Oct 29, 2026
Merged

helpcodeai merged 2 commits into
mainfrom
fix/adapter-params-1029

Conversation

@helpcodeai

Copy link
Copy Markdown
Contributor

Summary

Three adapter parameters that fail the same way every time in production (tool_invocations, read only, 28 to 29 Oct 2026). All three are schema or description fixes; no engine change.

Changes

  • Etsy etsy_get_listing, etsy_get_listings_by_ids: includes gets a pattern with the 7 values Etsy accepts. Models kept sending Inventory and got 400 every time: 22 errors in 6 workspaces in 24 h, 51 in 14 days, no success. The description now points to etsy_read_listing_inventory.
  • Telegram telegram_bot_get_updates: timeout capped at 25 s (minimum 0, maximum 25). The tool allowed 50 s, but the REST engine cuts a call at 30 s: 13 aborts in 4 workspaces in 24 h, 24 in 8 in 14 days.
  • Odoo odoo_call_method: the ids hint names message_post among the methods that need one id per call. "Expected singleton" came back 8 times in 3 workspaces after Fix Printify uploads limit, weclapp comment visibility, Odoo call_method hints #1311, all on message_post with several ids.

Type

  • Bug fix
  • New feature
  • Documentation
  • Refactoring
  • Performance improvement
  • Other (describe below)

Testing

  • Added/updated unit tests
  • Tested manually (describe steps below)
  • No tests needed (explain why)

Static assertions in etsy.live.spec.ts, telegram-bot.live.spec.ts and odoo.live.spec.ts (no live calls).

From the root:

  • node scripts/validate-adapters.mjs --warn: exit 0, no warning on the three files (the existing warnings are on other adapters).
  • node scripts/regenerate-catalog.mjs: no diff in catalog.ts.
  • node scripts/adapter-count.mjs --check: OK, 562 adapters.

In packages/backend, after npx prisma generate:

  • npm run lint: OK, no rewrites.
  • npx tsc --noEmit -p tsconfig.json: OK.
  • npm test: 800 suites passed, 4 skipped; 23,693 tests passed, 1,625 skipped.

Checklist

  • My code follows the existing code style
  • I have tested my changes locally
  • I have updated documentation (if applicable)
  • All existing tests pass (npm test)
  • This PR has a descriptive title

Related Issues

Follows #1311.

@helpcodeai
helpcodeai requested a review from D3nisty as a code owner October 29, 2026 08:25
@helpcodeai
helpcodeai enabled auto-merge (squash) October 29, 2026 08:25
@helpcodeai
helpcodeai merged commit aae3cc3 into main Oct 29, 2026
14 checks passed
@helpcodeai
helpcodeai deleted the fix/adapter-params-1029 branch October 29, 2026 08:35
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant