Skip to content

Feat/chutes provider integration#2

Open
Veightor wants to merge 30 commits intomainfrom
feat/chutes-provider-integration
Open

Feat/chutes provider integration#2
Veightor wants to merge 30 commits intomainfrom
feat/chutes-provider-integration

Conversation

@Veightor
Copy link
Collaborator

@Veightor Veightor commented Mar 6, 2026

Summary

Describe the problem and fix in 2–5 bullets:

  • Problem:
  • Why it matters:
  • What changed:
  • What did NOT change (scope boundary):

Change Type (select all)

  • Bug fix
  • Feature
  • Refactor
  • Docs
  • Security hardening
  • Chore/infra

Scope (select all touched areas)

  • Gateway / orchestration
  • Skills / tool execution
  • Auth / tokens
  • Memory / storage
  • Integrations
  • API / contracts
  • UI / DX
  • CI/CD / infra

Linked Issue/PR

  • Closes #
  • Related #

User-visible / Behavior Changes

List user-visible changes (including defaults/config).
If none, write None.

Security Impact (required)

  • New permissions/capabilities? (Yes/No)
  • Secrets/tokens handling changed? (Yes/No)
  • New/changed network calls? (Yes/No)
  • Command/tool execution surface changed? (Yes/No)
  • Data access scope changed? (Yes/No)
  • If any Yes, explain risk + mitigation:

Repro + Verification

Environment

  • OS:
  • Runtime/container:
  • Model/provider:
  • Integration/channel (if any):
  • Relevant config (redacted):

Steps

Expected

Actual

Evidence

Attach at least one:

  • Failing test/log before + passing after
  • Trace/log snippets
  • Screenshot/recording
  • Perf numbers (if relevant)

Human Verification (required)

What you personally verified (not just CI), and how:

  • Verified scenarios:
  • Edge cases checked:
  • What you did not verify:

Compatibility / Migration

  • Backward compatible? (Yes/No)
  • Config/env changes? (Yes/No)
  • Migration needed? (Yes/No)
  • If yes, exact upgrade steps:

Failure Recovery (if this breaks)

  • How to disable/revert this change quickly:
  • Files/config to restore:
  • Known bad symptoms reviewers should watch for:

Risks and Mitigations

List only real risks for this PR. Add/remove entries as needed. If none, write None.

  • Risk:
    • Mitigation:

Veightor added 10 commits March 6, 2026 15:11
- Correct chutes-fast alias to point to a valid model (GLM-4.7-FP8)
- Align setChutesApiKey signature with other providers (accept SecretInput)
- Use maybeSetResolvedApiKey in non-interactive Chutes onboarding
- Fix test mock data to match ChutesModelEntry API shape

Made-with: Cursor
…t detections)

- Fix Feishu media download/upload type errors by removing redundant timeout parameter (handled by client)
- Add 'pragma: allowlist secret' to false-positive test tokens and translations

Made-with: Cursor
Replace complex state-mutation helper with direct applyChutesConfig call after OAuth success, matching the pattern used by the API key flow. This reliably sets the Chutes default model so the model picker shows a Chutes model as current instead of falling back to the previous anthropic default.

Made-with: Cursor
The timeout parameter was removed from the source code in a previous commit
because it was causing type errors with the SDK, but the tests were not updated.
This brings the tests in sync with the source.

Made-with: Cursor
Conflict resolutions:
- src/secrets/target-registry-pattern.ts: adopt upstream's named-variable
  style for sibling_ref check; preserve pragma allowlist comment
- extensions/feishu/src/media.test.ts: keep upstream's
  expectMediaTimeoutClientConfigured() and add PR's imageCreateMock
  assertion for image_type: "message" verification
- src/agents/models-config.providers.ts: adopt upstream's refactored
  provider-loader pattern (SIMPLE/PROFILE_IMPLICIT_PROVIDER_LOADERS);
  wire Chutes into both API-key (withApiKey) and OAuth profile
  (withProfilePresence) loader slots using buildChutesProvider from
  models-config.providers.discovery.ts

Supporting changes to integrate Chutes into the new architecture:
- src/agents/models-config.providers.discovery.ts: add buildChutesProvider
  (dynamic model discovery via discoverChutesModels with access token)
- src/agents/model-auth-markers.ts: add CHUTES_OAUTH_MARKER and register
  it in isNonSecretApiKeyMarker

Made-with: Cursor
@Veightor Veightor force-pushed the feat/chutes-provider-integration branch from 46b18de to 106db9b Compare March 9, 2026 19:43
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.

2 participants