Merge upstream changes and consolidate Z.AI adapter - #3

Closed
AlexMikhalev wants to merge 12 commits into
mainfrom
merge-upstream-20251103
Closed

Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom
merge-upstream-20251103

Conversation

@AlexMikhalev

@AlexMikhalevAlexMikhalev commented Nov 3, 2025

Copy link
Copy Markdown

Summary

  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports

Changes Made

  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans

Testing

✅ Compilation successful
✅ Z.AI API integration verified with live tests
✅ Code quality checks passed (clippy, fmt)

🤖 Generated with terraphim.ai

pdylanrossand others added 12 commits October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev

AlexMikhalev commented Jan 12, 2026

Copy link
Copy Markdown
Author

PR #3 completed successfully!

All changes have been merged to main:

  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)

Additionally, AWS Bedrock adapter integration has been squash-merged to main with:

  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved

✅ All objectives achieved

AlexMikhalev added a commit that referenced this pull request Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit that referenced this pull request Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit that referenced this pull request Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Merge upstream changes and consolidate Z.AI adapter - #3
Closed
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Closed
Merge upstream changes and consolidate Z.AI adapter#3
AlexMikhalev wants to merge 12 commits into
mainfrom 
merge-upstream-20251103
Conversation
@AlexMikhalev
@AlexMikhalevAlexMikhalev
commented
Nov 3, 2025

Copy link


Copy Markdown

Summary
  • Merged latest upstream changes from jeremychone/rust-genai
  • Consolidated duplicate Zai/ZAi adapters into single Zai adapter
  • Fixed adapter import paths and module exports
Changes Made
  • Resolved merge conflicts in Z.AI adapter implementation
  • Updated model routing logic for Z.AI models
  • Removed obsolete zhipu adapter references
  • Added missing cerebras and openrouter module exports
  • Preserved dual endpoint support for Z.AI coding plans
Testing
✅ Compilation successful

✅ Z.AI API integration verified with live tests

✅ Code quality checks passed (clippy, fmt)
🤖 Generated with terraphim.ai
pdylanrossand others
added 12 commits
October 19, 2025 18:23
@pdylanross
added ModelName partial eq implementations for string types (jeremych…
6b3d3e0
…one#94)
@bartCarroll
Fixed streaming tool calls for openai models (jeremychone#91)
d3fcf0b
@jeremychone
- stream tool - openai - fix issue that captured_tool_calls ado not h…
4123ee2
…ave the fn_arguments parsed
@shingonoide
Refactor ZHIPU adapter to ZAI with namespace-based endpoint routing (j…
5a1df3c
…eremychone#95)
- Rename zhipu adapter to zai adapter
- Implement namespace-based routing for regular vs coding endpoints
- Regular models: "glm-4.6" → credit-based API
- Coding models: "zai::glm-4.6" → subscription API
- Add comprehensive example c07-zai.rs
- Update adapter kind detection to handle "zai" namespace
- Remove dual-endpoint complexity from user-facing API
Co-authored-by: Rui Andrada <rui.andrada@imaginedone.com.br>
@jeremychone
. anthropic - update model name for haiku 4.5
4aef5ca
@jeremychone
. anthropic - update claude-haiku(4.5) max tokens to 64k as per spec
d234513
@jeremychone
. first pass at adding the module spec
db19d5d
@jeremychone
. cargo fmt
28011fd
@jeremychone
. update to v0.4.3
1181667
@jeremychone
. v0.4.4-WIP
7b30e42
@AlexMikhalev@claude
Merge upstream changes into fork
1905659
Resolved conflicts in Z.AI adapter implementation:
- Consolidated duplicate Zai/ZAi adapters into single Zai adapter
- Updated model routing logic for Z.AI models
- Removed obsolete zhipu adapter references
- Added cerebras module to adapters
- Preserved dual endpoint support for Z.AI coding plans
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev@claude
fix: Clean up adapter imports and remove duplicate ZAi references
0f8839a
- Fixed import paths for adapters after module reorganization
- Removed remaining duplicate ZAi adapter references
- Added missing openrouter module export
- Ensured consistent adapter import structure
🤖 Generated with [terraphim.ai](https://terraphim.ai)
Co-Authored-By: Claude <noreply@anthropic.com>
@AlexMikhalev
AlexMikhalev
commented
Jan 12, 2026

Copy link


Copy Markdown

Author
PR #3 completed successfully!
All changes have been merged to main:
  • Fixed clippy warnings in test_adapter_consistency.rs and test_verify_model_lists.rs
  • Fixed case mismatch (ZAi vs Zai) in test expectations
  • Fixed ZAI model resolution test (glm-3-turbo routing to Zhipu)
  • All tests passing
  • Code quality checks passed (clippy, fmt, build)
Additionally, AWS Bedrock adapter integration has been squash-merged to main with:
  • Bearer token authentication
  • Full chat completion and streaming support
  • Tool/function calling support
  • Support for 26 Bedrock models
  • All clippy warnings resolved
✅ All objectives achieved
AlexMikhalev added a commit
that referenced
this pull request
Jan 12, 2026
@AlexMikhalev@claude
docs: add handover document and lessons learned from PR#3fix
baac9b6
Add comprehensive documentation from session fixing PR #3 and merging
Bedrock integration:
HANDOVER.md:
- Complete progress summary (PR #3 fixed, Bedrock integrated)
- Technical context (commits, files changed, discoveries)
- Next steps and verification commands
- All 4 commits documented with details
lessons-learned.md:
- 6 technical discoveries (Debug format, routing, let-chains, etc.)
- Debugging approaches that worked
- 5 pitfalls to avoid with solutions
- 5 best practices discovered
- Quick reference for commands and file locations
Generated after successful completion of:
- Fixed clippy warnings in test files and Bedrock adapter
- Fixed ZAI model routing test expectations
- Squash-merged AWS Bedrock adapter integration
- All CI checks passing, main branch stable
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
symphony:#3- Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
7d99889
AlexMikhalev added a commit
that referenced
this pull request
Mar 16, 2026
@AlexMikhalev
Merge#3: Update Cerebras adapter for v0.6.0-beta.8 Adapter trait
41ec47c
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

5 participants

@AlexMikhalev@pdylanross@bartCarroll@jeremychone@shingonoide