Skip to content

Document all supported Jev providers - #6

Open
dajiaohuang wants to merge 1 commit into
kitze:mainfrom
dajiaohuang:fix-openrouter-provider-docs
Open

dajiaohuang wants to merge 1 commit into
kitze:mainfrom
dajiaohuang:fix-openrouter-provider-docs

Conversation

@dajiaohuang

@dajiaohuang dajiaohuang commented Sep 23, 2026 •

Copy link
Copy Markdown

Summary: Update self-hosting and Umbrel descriptions to include OpenRouter, and add a regression assertion for the generated package description. Validation passed: bun run typecheck; bun run build (existing large-chunk warning); focused self-hosting tests (5 tests, 47 assertions); git diff --check. Database-backed tests were unavailable because Docker was unavailable. The broader suite also showed separate Windows file-mode and package-materialization EPERM failures, tracked independently.

Summary by CodeRabbit

  • Documentation
    • Updated setup and self-hosting guidance to list OpenRouter alongside TypeSafe AI and Vercel AI Gateway as an optional provider for Jev recommendations.
    • Clarified that provider keys are configured in Skillbox Settings, not as image or global deployment keys. The Executor requirement remains unchanged.
    • Updated the Umbrel app description to mention OpenRouter.

@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 14596643-b0ec-4e28-a57e-ba097bfd9f53

📥 Commits

Reviewing files that changed from the base of the PR and between cda64ad and 49bf550.

📒 Files selected for processing (5)
  • deploy/umbrel/README.md
  • docs/open-source-readiness.md
  • docs/self-hosting.md
  • scripts/package-umbrel.ts
  • tests/self-hosting.test.ts

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

Documentation and the generated Umbrel manifest now list OpenRouter as an optional provider for Jev. A test asserts that the generated manifest description contains “OpenRouter.”

Changes

Jev provider references

Layer / File(s) Summary
Document and validate the OpenRouter provider reference
docs/open-source-readiness.md, docs/self-hosting.md, deploy/umbrel/README.md, scripts/package-umbrel.ts, tests/self-hosting.test.ts
Documentation and the generated Umbrel manifest description add OpenRouter to the Jev provider options. The test checks that the description contains “OpenRouter.”

Estimated code review effort: 2 (Simple) | ~8 minutes

Suggested reviewers: kitze

Merge Risk: ⚪ Minimal · up to 49bf5

The self-hosted Jev integration supports OpenRouter with a key saved in Settings, and the Umbrel description reflects that behavior. No user-facing regression is identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (3 skipped: 3 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: documenting the supported Jev providers, including OpenRouter, across self-hosting and Umbrel documentation.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (3 skipped: 3 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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.

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