Repository navigation
chore(skills): migrate opencode-lucky-workflows to Claude-lucky-workflows, fix SSH refs - #289
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThis PR updates workflow documentation by adding Claude Code Lucky Workflows skill documentation, removing OpenCode Lucky Workflows documentation, updating SSH target hosts in the lucky-deploy-recovery workflow, and replacing OpenCode references with Claude Code references in AGENTS.md. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
✅ Deploy Preview for regal-bunny-0c8efe ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
LucasSantana-Dev
force-pushed
the
chore/skills-maintenance
branch
from
March 15, 2026 21:50
46674ef to
2548e0e
Compare
LucasSantana-Dev
force-pushed
the
chore/skills-maintenance
branch
from
March 15, 2026 21:52
2548e0e to
1897784
Compare
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.cursor/skills/lucky-deploy-recovery/SKILL.md (1)
33-40: Host targeting is now inconsistent with existing automation defaults.This doc now hardcodes
luk-server@100.95.204.103, while existing helper scripts still default toserver-do-luk. Consider documenting one canonical host selector (alias or explicit host) plus a fallback note to avoid split-brain runbooks.
Cross-file evidence:scripts/homelab-diagnostics.sh:4,scripts/opencode-attach-server-do-luk.sh:5,scripts/opencode-sync-server-do-luk-skills.sh:6.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.cursor/skills/lucky-deploy-recovery/SKILL.md around lines 33 - 40, The doc hardcodes the host literal "luk-server@100.95.204.103" causing divergence from existing automation that uses the "server-do-luk" alias; update the SKILL.md steps that use "luk-server@100.95.204.103" to reference a single canonical host selector (prefer the alias "server-do-luk") and add a one-line fallback note showing the explicit IP form for manual use; ensure the change touches the commands shown (the ssh lines and the stash/reset sequence) and mention the helper scripts (scripts/homelab-diagnostics.sh, scripts/opencode-attach-server-do-luk.sh, scripts/opencode-sync-server-do-luk-skills.sh) as the canonical source of truth so users know which selector to use.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@AGENTS.md`:
- Around line 149-154: AGENTS.md references four scripts that do not exist
(scripts/Claude-sync-project-skills.sh,
scripts/Claude-install-community-plugins.sh, scripts/Claude-verify.sh,
scripts/Claude-attach-server-do-luk.sh); update the doc to either remove these
script references or add/implement the missing scripts. Locate the mentions in
AGENTS.md (the block listing Repo-local Claude Code and the four scripts) and
either delete the four script lines and replace them with equivalent manual
steps or create the corresponding executable scripts that perform the described
tasks (sync project skills, install community plugins, verify config/plugins,
and attach to server with Claude_REMOTE_DIR support), ensuring names exactly
match the referenced symbols so the documentation and repo stay consistent.
---
Nitpick comments:
In @.cursor/skills/lucky-deploy-recovery/SKILL.md:
- Around line 33-40: The doc hardcodes the host literal
"luk-server@100.95.204.103" causing divergence from existing automation that
uses the "server-do-luk" alias; update the SKILL.md steps that use
"luk-server@100.95.204.103" to reference a single canonical host selector
(prefer the alias "server-do-luk") and add a one-line fallback note showing the
explicit IP form for manual use; ensure the change touches the commands shown
(the ssh lines and the stash/reset sequence) and mention the helper scripts
(scripts/homelab-diagnostics.sh, scripts/opencode-attach-server-do-luk.sh,
scripts/opencode-sync-server-do-luk-skills.sh) as the canonical source of truth
so users know which selector to use.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: ef335c93-e0cc-4d92-8ae2-5bc86217633e
📒 Files selected for processing (4)
.cursor/skills/Claude-lucky-workflows/SKILL.md.cursor/skills/lucky-deploy-recovery/SKILL.md.cursor/skills/opencode-lucky-workflows/SKILL.mdAGENTS.md
💤 Files with no reviewable changes (1)
- .cursor/skills/opencode-lucky-workflows/SKILL.md
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Quality Gates
- GitHub Check: SonarCloud Scan
🧰 Additional context used
🧠 Learnings (15)
📓 Common learnings
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-data.mdc:0-0
Timestamp: 2026-03-09T20:21:46.291Z
Learning: Use `.cursor/skills/prisma-redis-lucky/SKILL.md` for Prisma schema, migrations, Redis client, and key patterns
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to .opencode/skills/**/* : Store OpenCode skill bridging in `.opencode/skills`; use `scripts/opencode-sync-project-skills.sh` after changes
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: When CI tests fail or gate checks are required, use the `lucky-ci-gate-recovery` skill for triage and recovery; when deployment issues occur, use `lucky-deploy-recovery` and `deploy-homelab` skills for webhook errors and container recovery
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to **/*.{ts,tsx} : Use the `prisma-redis-lucky` skill when adding or modifying database schema, migrations, or Redis operations in the shared package
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to **/Dockerfile : Use the `lucky-docker-dev` skill for Docker container configuration, build, and deployment
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to docs/**/*.md : Update relevant documentation in `docs/` (ARCHITECTURE, MCP_SETUP, FRONTEND, WEBAPP_SETUP, etc.) when behavior or setup changes; use `mcp-docs-search` skill for docs lookup and web search
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-09T20:20:32.245Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/documentation.mdc:0-0
Timestamp: 2026-03-09T20:20:32.245Z
Learning: Applies to README.md : Update README.md if behavior changed
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:20:32.245Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/documentation.mdc:0-0
Timestamp: 2026-03-09T20:20:32.245Z
Learning: Applies to **/docs/**/*.{md,mdx} : Keep API documentation in sync with code changes
Applied to files:
AGENTS.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: When CI tests fail or gate checks are required, use the `lucky-ci-gate-recovery` skill for triage and recovery; when deployment issues occur, use `lucky-deploy-recovery` and `deploy-homelab` skills for webhook errors and container recovery
Applied to files:
AGENTS.md.cursor/skills/lucky-deploy-recovery/SKILL.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to opencode.jsonc : OpenCode behavior configuration lives in `opencode.jsonc`, `.opencode/plugins`, and `.opencode/skills`; use `scripts/opencode-sync-project-skills.sh` after changing project skills or OpenCode skill bridges, `scripts/opencode-install-community-plugins.sh` to prime approved plugins, and `scripts/opencode-verify.sh` after config/plugin changes
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Use the `e2e-playwright` skill for E2E tests with Playwright and browser verification; use `testing-lucky` skill for unit tests, Jest ESM mocks, and enabling disabled tests
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to .opencode/skills/**/* : Store OpenCode skill bridging in `.opencode/skills`; use `scripts/opencode-sync-project-skills.sh` after changes
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to .opencode/plugins/**/* : Store OpenCode plugin configurations in `.opencode/plugins`; use `scripts/opencode-sync-project-skills.sh` after changes and `scripts/opencode-verify.sh` to validate
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-09T20:21:46.291Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-data.mdc:0-0
Timestamp: 2026-03-09T20:21:46.291Z
Learning: Use `.cursor/skills/prisma-redis-lucky/SKILL.md` for Prisma schema, migrations, Redis client, and key patterns
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to docker-compose.yml : Use the `lucky-docker-dev` skill for Docker and docker-compose configuration, local development setup, and container management
Applied to files:
AGENTS.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-09T20:21:08.612Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/lucky-project.mdc:0-0
Timestamp: 2026-03-09T20:21:08.612Z
Learning: Applies to docker-compose.dev.yml : Use Docker for local environment setup when available via `docker-compose.dev.yml`
Applied to files:
AGENTS.md
📚 Learning: 2026-03-15T14:34:47.397Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-15T14:34:47.397Z
Learning: Applies to **/Dockerfile : Use the `lucky-docker-dev` skill for Docker container configuration, build, and deployment
Applied to files:
AGENTS.md.cursor/skills/lucky-deploy-recovery/SKILL.md.cursor/skills/Claude-lucky-workflows/SKILL.md
📚 Learning: 2026-03-09T20:22:25.255Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/workflow.mdc:0-0
Timestamp: 2026-03-09T20:22:25.255Z
Learning: Applies to {**/scripts/**,scripts/**,.github/workflows/**/*.{yml,yaml},**/.gitlab-ci.yml,.circleci/config.yml} : Use cross-platform environment handling in scripts, cross-platform deletion utilities instead of OS-specific commands, pass non-interactive flags (--yes, --ci) by default in automation, and avoid OS-specific commands
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:08.612Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/lucky-project.mdc:0-0
Timestamp: 2026-03-09T20:21:08.612Z
Learning: Applies to {scripts/**,packages/scripts/**} : Prefer `packages/scripts` or root `scripts/` for one-off automation; avoid maintaining one-time scripts in the repository long term
Applied to files:
AGENTS.md
🪛 LanguageTool
AGENTS.md
[style] ~153-~153: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...dd-ons locally or on server-do-luk. - Use scripts/Claude-verify.sh after Claude...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
🔇 Additional comments (2)
AGENTS.md (1)
41-41: Rename in skills table looks consistent.This row correctly aligns the table with the new
Claude-lucky-workflowsskill naming..cursor/skills/Claude-lucky-workflows/SKILL.md (1)
1-110: New Claude workflow skill doc is clear and well-scoped.Scope, config boundaries, validation flow, and remote recovery guidance are documented in a usable sequence.
|
LucasSantana-Dev
added a commit
that referenced
this pull request
May 13, 2026
…lows, fix SSH refs (#289) * chore(skills): migrate opencode-lucky-workflows to Claude-lucky-workflows, fix SSH refs * fix(test): add prismaClient mock to prevent ESM import.meta parse error --------- Co-authored-by: Lucas Santana <lucassantana@users.noreply.github.com> Co-authored-by: Lucas Santana <lucas@lucassantana.tech>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
opencode-lucky-workflowsskill toClaude-lucky-workflows— the project now uses Claude Code, not OpenCode. Rewrote the skill to referenceClaude.jsonc,.Claude/plugins,.Claude/skills, andscripts/Claude-*scripts.lucky-deploy-recovery/SKILL.md— replaced stalessh server-do-lukwithssh luk-server@100.95.204.103(Tailscale) in triage sequence.AGENTS.md— table row now points toClaude-lucky-workflows, bottom section referencesClaude.jsonc/.Claude//scripts/Claude-*instead of OpenCode equivalents.No code changes — skills and AGENTS.md only.
Summary by CodeRabbit
Documentation
Chores