Skip to content

feat(kontext): support user-token runtime connect type (sdk 0.2.0)#29

Merged
michiosw merged 2 commits into
kontext-devfrom
feat/kontext-user-token-cutover
Mar 4, 2026
Merged

feat(kontext): support user-token runtime connect type (sdk 0.2.0)#29
michiosw merged 2 commits into
kontext-devfrom
feat/kontext-user-token-cutover

Conversation

@michiosw
Copy link
Copy Markdown

@michiosw michiosw commented Mar 4, 2026

Summary

  • bump kontext-dev-sdk workspace dependency to 0.2.0
  • update lockfiles (Cargo.lock, MODULE.bazel.lock) for the SDK/core cutover
  • add explicit RuntimeIntegrationConnectType handling for disconnected integrations in kontext_dev runtime
  • treat UserToken as per-user token/API key requirement in user guidance logs
  • add tests to validate runtime integrations payload parsing for:
    • connectType: "user_token" accepted
    • unknown connectType rejected as invalid payload

Validation

  • cargo test -p codex-core kontext_dev::tests
  • just bazel-lock-check

Open with Devin

@michiosw michiosw enabled auto-merge (squash) March 4, 2026 13:36
Copy link
Copy Markdown

@kontext-review kontext-review Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Kontext Review: 1 Issues Found (Advisory)

Inline findings posted:

  • [LOW][testing] Architectural finding (codex-rs/core/src/kontext_dev.rs:794)

Reviewer summary:

  • Bumped kontext-dev-sdk to 0.2.0 and updated lockfiles.
  • Added explicit handling for RuntimeIntegrationConnectType to provide better user guidance for disconnected integrations.
  • Added tests to verify payload parsing for the new user_token connect type.
Scores and diagnostics
  • Architecture: 7.0/10
  • Organization: 7.0/10
  • Cleanliness: 7.0/10
  • Overall: 7.0/10
  • Candidate findings: 1
  • Published inline: 1
  • Published summary-only: 0
  • Suppressed findings: 0

Limitations

  • Cannot verify the exact structure of kontext_dev_sdk::mcp::RuntimeIntegrationRecord as it is in an external/workspace crate not included in the diff.

  • Model: gemini-3.1-pro-preview

  • Omitted files: 0

Generated at 2026-03-04T13:37:05.810Z.

Comment thread codex-rs/core/src/kontext_dev.rs
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

Copy link
Copy Markdown

@kontext-review kontext-review Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Kontext Review: 1 Issues Found (Advisory)

Inline findings posted:

  • [LOW][maintainability] Architectural finding (codex-rs/Cargo.toml:99)

Reviewer summary:

  • Bumped kontext-dev-sdk to 0.2.0 and updated lockfiles.
  • Added handling for RuntimeIntegrationConnectType to provide more specific user guidance when integrations are disconnected.
  • Added tests to verify the parsing of the new connectType field in runtime integration payloads.
  • Added a new codex binary alias in codex-rs/cli/Cargo.toml.
Scores and diagnostics
  • Architecture: 7.0/10

  • Organization: 7.0/10

  • Cleanliness: 7.0/10

  • Overall: 7.0/10

  • Candidate findings: 1

  • Published inline: 1

  • Published summary-only: 0

  • Suppressed findings: 0

  • Model: gemini-3.1-pro-preview

  • Omitted files: 0

Generated at 2026-03-04T14:05:05.568Z.

Comment thread codex-rs/Cargo.toml
@michiosw michiosw merged commit ce4dbcb into kontext-dev Mar 4, 2026
15 checks passed
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