Skip to content

feat: add DeepSeek native implementer lane - #13

Open
bububa wants to merge 10 commits into
DannyMac180:mainfrom
bububa:main
Open

feat: add DeepSeek native implementer lane#13
bububa wants to merge 10 commits into
DannyMac180:mainfrom
bububa:main

Conversation

@bububa

@bububa bububa commented Aug 6, 2026

Copy link
Copy Markdown

Summary

  • Add the sol_advisor_deepseek_implementer native role pinned to deepseek/deepseek-v4-flash with high reasoning.
  • Install and verify DeepSeek, Terra, and Sol companion roles by default while preserving the retired Luna companion migration.
  • Add a capability-gated DeepSeek native-lane contract with explicit pre-work fallback to Terra only for automatic selection; explicit DeepSeek requests fail closed.
  • Preserve fresh sol_advisor_sol_reviewer review for every successful native implementation.
  • Update orchestration documentation, plugin metadata, UI text, and version to 0.5.0.

Validation

  • sh plugins/sol-advisor/scripts/verify.sh — passed.
  • Official skill validation — passed.
  • Official plugin validation — passed.
  • jq empty .agents/plugins/marketplace.json plugins/sol-advisor/.codex-plugin/plugin.json — passed.
  • git diff --check — passed.

Routing notes

  • The DeepSeek role is installed by default but does not start or configure OpenCodex.
  • The native lane uses the provider's default multi-agent mode when the exact DeepSeek route is advertised.
  • Terra remains available when DeepSeek is unavailable; no fallback occurs after worker execution begins or for an explicitly requested DeepSeek lane.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 62f210e455

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread plugins/sol-advisor/agents/sol-advisor-terra-implementer.toml Outdated
Comment thread plugins/sol-advisor/.codex-plugin/plugin.json Outdated
bububa and others added 8 commits August 6, 2026 15:54
Bring DannyMac180/sol-advisor main (cross-client conformance, Cursor MCP
bridge, bun validation) into the fork and re-integrate the PR's
capability-gated DeepSeek native lane into the new architecture. Resolves
conflicts in README.md, .codex-plugin/plugin.json, verify.sh, and
orchestration SKILL.md, preserving both the DeepSeek lane and the upstream
conformance structure.
@bububa

bububa commented Aug 17, 2026

Copy link
Copy Markdown
Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1792547e7e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread plugins/sol-advisor/skills/orchestration/references/operations.md Outdated
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