Skip to content

feat(ai): delegate product defaults to adaptive orchestration - #96

Closed
seonghobae wants to merge 7 commits into
mainfrom
agent/adaptive-orchestrator-default
Closed

feat(ai): delegate product defaults to adaptive orchestration#96
seonghobae wants to merge 7 commits into
mainfrom
agent/adaptive-orchestrator-default

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Summary

  • replace production mode="route" defaults in summarization, extraction, classification, chat, commitment, and post-evaluation adapters with contextual-orchestrator auto;
  • preserve the explicit adjudication verify contract;
  • add regressions that inspect every runtime adapter and exercise the post-evaluation transport default;
  • document the ownership boundary and APA 7 research basis in ADR-0005 and CHANGELOG.

A temporary branch workflow stages the regression before applying source changes, proves the red/green transition when needed, runs the complete suite, checks syntax and patch integrity, removes itself, and publishes only the verified source commit.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a66613e0-92d9-44b5-abc2-834140964b12


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Competing with #91 on the same adaptive-orchestrator default. Both collide with #74/#89 ADR and version numbers. Leave blocked; do not merge first. After #74+#89, keep one of #91/#96 and rebase as ADR 0015 / 0.80.x.

- .github/workflows/apply-adaptive-orchestrator-default.yml

permissions:
contents: write
shell: bash
run: |
set -euo pipefail
python -m pip install --upgrade pip
run: |
set -euo pipefail
python -m pip install --upgrade pip
python -m pip install -e '.[dev,backend]'
@github-actions
github-actions Bot disabled auto-merge August 16, 2026 12:39
@seonghobae
seonghobae enabled auto-merge (squash) August 16, 2026 12:52
@seonghobae

Copy link
Copy Markdown
Contributor Author

Superseded by #91 (c93d449) on main. Remaining unique delta is a post_chat docstring (route vs auto); runtime consumers already default to auto/verify as decided in ADR-0013. Closing to clear the conflicting queue.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Closing as superseded by #91.

@seonghobae seonghobae closed this Aug 16, 2026
auto-merge was automatically disabled August 16, 2026 14:09

Pull request was closed

Copy link
Copy Markdown
Contributor Author

Confirming #96 stays closed unmerged. Adaptive defaults already landed on main via #91.

Copy link
Copy Markdown
Contributor Author

Cycle retry 2026-08-17: #96 remains closed as weaker duplicate of merged #91. No reopen.

Copy link
Copy Markdown
Contributor Author

Cycle retry 2026-08-17: closing as weaker duplicate of #91 (ADR 0015). Independent review still required on exact heads; this is not an approval.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-18T16:05 KST: weaker duplicate of merged #91. Closing remains the intended disposition. Independent APPROVE of #74/#92 is still required; this comment is not an approval.

Copy link
Copy Markdown
Contributor Author

Cycle retry: close weaker duplicate of #91. Independent review still required elsewhere. Do not self-approve.

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