Skip to content

feat(skiller): support external Skiller web service#379

Merged
skulidropek merged 4 commits into
mainfrom
issue-376
Jun 6, 2026
Merged

feat(skiller): support external Skiller web service#379
skulidropek merged 4 commits into
mainfrom
issue-376

Conversation

@skulidropek

@skulidropek skulidropek commented Jun 6, 2026

Copy link
Copy Markdown
Member

Summary

  • add external Skiller Web mode via DOCKER_GIT_SKILLER_WEB_URL
  • return /launch URLs with backendUrl, projectKey, and sessionId instead of launching bundled Electron when external mode is configured
  • expose project/session Skiller context endpoints for Skiller Web browser scope
  • support production Skiller Web -> local docker-git CORS, including Chrome Private Network Access preflight
  • keep legacy bundled Skiller behavior when external URL is unset

Closes #376
Skiller Web merged PRs:

Verification

  • bun run --filter @effect-template/api lint
  • bun run --filter @effect-template/api typecheck
  • bun run --filter @effect-template/api test
  • bun test packages/api/tests/skiller-core.test.ts packages/api/tests/skiller-cors.test.ts packages/api/tests/skiller-routes.test.ts
  • GitHub Actions for commit 2d3fe06: Snapshot, Checking Dependencies, Final Build, and Check all passed
  • Vercel production deploy: https://skiller-web-henna.vercel.app
  • Browser smoke: production /#/?backendUrl=http%3A%2F%2F127.0.0.1%3A4181 auto-loaded projects from mock docker-git backend
  • Browser smoke: production /launch?backendUrl=http%3A%2F%2F127.0.0.1%3A4181&projectKey=project-proof auto-connected and stored docker-git connection

@coderabbitai

coderabbitai Bot commented Jun 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@skulidropek, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 49 minutes and 6 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 931240a2-d589-43d4-bca0-1159bf514f05

📥 Commits

Reviewing files that changed from the base of the PR and between b4426a3 and 2d3fe06.

📒 Files selected for processing (8)
  • packages/api/src/http.ts
  • packages/api/src/services/skiller-core.ts
  • packages/api/src/services/skiller.ts
  • packages/api/tests/skiller-core.test.ts
  • packages/api/tests/skiller-cors.test.ts
  • packages/api/tests/skiller-routes.test.ts
  • packages/app/src/web/actions-skiller.ts
  • packages/app/src/web/api-skiller-schema.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue-376

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 and usage tips.

@skulidropek

skulidropek commented Jun 6, 2026

Copy link
Copy Markdown
Member Author

AI Session Backup

Commit: 670a168
Status: success
Files: 8 (12.47 MB)
Links: README | Manifest

git status

On branch issue-376
Your branch is up to date with 'origin/issue-376'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
  (commit or discard the untracked or modified content in submodules)
	modified:   third_party/skiller-desktop-skills-manager (modified content)

no changes added to commit (use "git add" and/or "git commit -a")

@skulidropek

skulidropek commented Jun 6, 2026

Copy link
Copy Markdown
Member Author

AI Session Backup

Commit: dcdd8c1
Status: success
Files: 9 (17.83 MB)
Links: README | Manifest

git status

On branch issue-376
Your branch is up to date with 'origin/issue-376'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
  (commit or discard the untracked or modified content in submodules)
	modified:   third_party/skiller-desktop-skills-manager (modified content)

no changes added to commit (use "git add" and/or "git commit -a")

@skulidropek

skulidropek commented Jun 6, 2026

Copy link
Copy Markdown
Member Author

AI Session Backup

Commit: 2d3fe06
Status: success
Files: 9 (17.96 MB)
Links: README | Manifest

git status

On branch issue-376
Your branch is up to date with 'origin/issue-376'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
  (commit or discard the untracked or modified content in submodules)
	modified:   third_party/skiller-desktop-skills-manager (modified content)

no changes added to commit (use "git add" and/or "git commit -a")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant