Repository navigation
docs(skills): capture deploy and MCP recovery playbooks - #248
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for regal-bunny-0c8efe ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
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 (4)
📝 WalkthroughWalkthroughThis pull request adds a new Lucky deployment recovery skill guide and expands existing skill documentation. It updates GitHub MCP recovery guidance to use the official github-mcp-server binary instead of the npm runtime, and documents this migration across setup guides and AGENTS references. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.cursor/skills/mcp-github-recovery/SKILL.md:
- Line 110: The wrapper script creation fails if the target directory doesn't
exist; before the heredoc that writes "run-mcp-github.sh" (the line starting
with cat > "$HOME/.codex/scripts/run-mcp-github.sh" <<'EOF'), add a step to
create the wrapper directory (ensure the "$HOME/.codex/scripts" directory is
created with parents) and apply the same change to the second occurrence that
also writes into the .codex/scripts directory.
- Around line 115-118: The current token resolution sets token from environment
first (token="${GITHUB_PERSONAL_ACCESS_TOKEN:-${GITHUB_TOKEN:-}}") and falls
back to gh auth token; change the precedence to call gh auth token first and
only use the env vars as a fallback so the wrapper matches the documented
policy—i.e., attempt capturing gh auth token (via the same gh auth token command
and assignment used now), and if that yields empty, then assign from
GITHUB_PERSONAL_ACCESS_TOKEN or GITHUB_TOKEN; update the logic around the token
variable and the if/command -v gh block accordingly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4b5ae9cf-3548-4603-84bf-2db02c9eb42e
📒 Files selected for processing (7)
.cursor/skills/lucky-deploy-recovery/SKILL.md.cursor/skills/lucky-docker-dev/SKILL.md.cursor/skills/mcp-github-recovery/SKILL.mdAGENTS.mdCHANGELOG.mdREADME.mddocs/MCP_SETUP.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
📓 Path-based instructions (5)
{CHANGELOG.md,README.md}
📄 CodeRabbit inference engine (.cursor/rules/agent-rules.mdc)
ALWAYS update CHANGELOG.md and README.md as changes are made.
Files:
README.mdCHANGELOG.md
README.md
📄 CodeRabbit inference engine (.cursor/rules/templates-examples.mdc)
README.md must be updated if behavior changed
Update README.md if behavior changed
Files:
README.md
CHANGELOG.md
📄 CodeRabbit inference engine (.cursor/rules/templates-examples.mdc)
CHANGELOG.md must be updated with all changes in pull requests
Always update CHANGELOG.md with all code changes
Update CHANGELOG.md with all changes, include breaking changes documentation, and reference issues and PRs
Files:
CHANGELOG.md
{CHANGELOG.md,docs/**}
📄 CodeRabbit inference engine (.cursor/rules/lucky-project.mdc)
Update
CHANGELOG.mdand relevantdocs/files when behavior or setup changes
Files:
CHANGELOG.mddocs/MCP_SETUP.md
**/docs/**/*.{md,mdx}
📄 CodeRabbit inference engine (.cursor/rules/documentation.mdc)
**/docs/**/*.{md,mdx}: Keep API documentation in sync with code changes
Document significant architectural design choices
Files:
docs/MCP_SETUP.md
🧠 Learnings (24)
📓 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
📚 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:
.cursor/skills/lucky-deploy-recovery/SKILL.mdREADME.md.cursor/skills/lucky-docker-dev/SKILL.mdAGENTS.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:
README.md.cursor/skills/mcp-github-recovery/SKILL.mddocs/MCP_SETUP.md
📚 Learning: 2026-03-14T16:53:29.998Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-14T16:53:29.998Z
Learning: Update `CHANGELOG.md` and relevant `docs/` when behavior or setup changes
Applied to files:
README.mddocs/MCP_SETUP.md
📚 Learning: 2026-03-14T16:53:29.998Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-14T16:53:29.998Z
Learning: Use the appropriate project skill from `.cursor/skills/` based on the task type (e.g., `discord-commands` for slash commands, `music-queue-player` for player operations, `prisma-redis-lucky` for schema/migrations)
Applied to files:
README.md
📚 Learning: 2026-03-09T20:21:38.098Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-backend.mdc:0-0
Timestamp: 2026-03-09T20:21:38.098Z
Learning: Applies to packages/backend/**/*.ts : Use `.cursor/skills/backend-express/SKILL.md` for Express routes, middleware, and services when acting as backend specialist
Applied to files:
README.md
📚 Learning: 2026-03-09T20:21:52.065Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-discord.mdc:0-0
Timestamp: 2026-03-09T20:21:52.065Z
Learning: Applies to packages/bot/src/functions/{general,music,download}/commands/**/*.ts : Use `.cursor/skills/discord-commands/SKILL.md` for implementing slash commands
Applied to files:
README.md
📚 Learning: 2026-03-09T20:21:15.595Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/lucky-shared.mdc:0-0
Timestamp: 2026-03-09T20:21:15.595Z
Learning: Applies to packages/shared/**/*.ts : Use the single Prisma client located at `packages/shared/src/utils/database/prismaClient.ts`; maintain schema in repo root at `prisma/schema.prisma`; run migrations from root using `npm run db:migrate`
Applied to files:
CHANGELOG.md
📚 Learning: 2026-03-14T16:53:29.998Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-14T16:53:29.998Z
Learning: Use `npm run dev:bot`, `npm run dev:backend`, `npm run dev:frontend` for development; use Docker for local when available (`docker-compose.dev.yml`)
Applied to files:
.cursor/skills/lucky-docker-dev/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:
.cursor/skills/lucky-docker-dev/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 package.json : Use `npm run dev:bot`, `npm run dev:backend`, and `npm run dev:frontend` for development, and `npm run db:*` commands for database operations
Applied to files:
.cursor/skills/lucky-docker-dev/SKILL.mdAGENTS.md
📚 Learning: 2026-03-09T20:21:31.459Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/quality.mdc:0-0
Timestamp: 2026-03-09T20:21:31.459Z
Learning: Run production build verification in CI quality checks
Applied to files:
.cursor/skills/lucky-docker-dev/SKILL.md
📚 Learning: 2026-03-14T16:53:29.998Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-14T16:53:29.998Z
Learning: No hardcoded secrets, IPs, or ports; use `.env` and `docs/` for required environment variables
Applied to files:
.cursor/skills/lucky-docker-dev/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: PR checklist must verify: tests added/updated, lint and build pass, commits follow conventional format, scripts used where applicable, CHANGELOG.md updated, README.md updated if behavior changed, documentation in sync, and security scan completed
Applied to files:
.cursor/skills/lucky-docker-dev/SKILL.md
📚 Learning: 2026-03-09T20:21:38.098Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-backend.mdc:0-0
Timestamp: 2026-03-09T20:21:38.098Z
Learning: Prefer **user-Context7** MCP for Express, Node, TypeScript tasks when acting as backend specialist
Applied to files:
docs/MCP_SETUP.md
📚 Learning: 2026-03-14T16:53:29.998Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-03-14T16:53:29.998Z
Learning: Use ecosystem skills from `.agent-skills/` for cross-cutting concerns: `systematic-debugging` for bugs/test failures, `test-driven-development` for features/bugfixes, `verification-before-completion` before marking work complete, `brainstorming` before new features, `requesting-code-review` before merging
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:52.065Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-discord.mdc:0-0
Timestamp: 2026-03-09T20:21:52.065Z
Learning: Applies to packages/bot/src/functions/music/commands/**/*.ts : Use `.cursor/skills/music-queue-player/SKILL.md` for play, queue, skip, volume commands and player lifecycle management
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 package.json : Use npm workspaces monorepo structure with packages in `packages/*` directory containing: `shared` (config, services, types, utils), `bot` (Discord.js/Discord Player), `backend` (Express API), and `frontend` (React/Vite)
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:15.595Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/lucky-shared.mdc:0-0
Timestamp: 2026-03-09T20:21:15.595Z
Learning: Applies to packages/shared/**/*.ts : Organize the Lucky Shared Package with the following directory structure: Config in `packages/shared/src/config/` (environment, constants, feature toggles, YouTube config); Services in `packages/shared/src/services/` (DatabaseService, Redis client/operations, FeatureToggleService, ReactionRoles, RoleManagement); Types in `packages/shared/src/types/` (errors, commands, common, discord, music); Utils in `packages/shared/src/utils/` (error handling, retry, embeds, log, monitoring, composables, prismaClient)
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:20:47.877Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/lucky-discord-bot.mdc:0-0
Timestamp: 2026-03-09T20:20:47.877Z
Learning: Applies to packages/bot/**/*.{ts,tsx} : Do not duplicate queue or player state outside Discord Player; use shared services from `lucky/shared` for persistent data like track history and session information
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:38.098Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-backend.mdc:0-0
Timestamp: 2026-03-09T20:21:38.098Z
Learning: Applies to packages/backend/**/*.ts : Use `lucky/shared` for config and DB/Redis when needed in backend code
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:52.065Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-discord.mdc:0-0
Timestamp: 2026-03-09T20:21:52.065Z
Learning: Applies to packages/bot/src/**/*.ts : Use `lucky/shared` for database, Redis, logging, and embed utilities instead of implementing them locally
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:38.098Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-backend.mdc:0-0
Timestamp: 2026-03-09T20:21:38.098Z
Learning: Applies to packages/backend/**/*.ts : Apply `.cursor/rules/lucky-backend-api.mdc` for structure and conventions when acting as backend specialist
Applied to files:
AGENTS.md
📚 Learning: 2026-03-09T20:21:58.991Z
Learnt from: CR
Repo: LucasSantana-Dev/Lucky PR: 0
File: .cursor/rules/subagent-frontend.mdc:0-0
Timestamp: 2026-03-09T20:21:58.991Z
Learning: Prefer user-Context7 for React, Vite, Tailwind, and TypeScript documentation; use user-browser-tools or cursor-ide-browser for E2E testing; use user-v0 or user-magicuidesign/mcp for UI ideas only
Applied to files:
AGENTS.md
🔇 Additional comments (6)
CHANGELOG.md (1)
15-18: Good Unreleased changelog coverage.This entry clearly captures both deploy-recovery documentation and the GitHub MCP runtime migration for this PR scope.
.cursor/skills/lucky-deploy-recovery/SKILL.md (1)
1-48: Solid recovery runbook structure.The flow is clear, operationally safe, and emphasizes outcome verification over workflow-status-only checks.
AGENTS.md (1)
29-29: Nice skill-routing addition.The new task-to-skill mapping is precise and matches the new recovery skill intent.
README.md (1)
337-339: README runtime migration note is clear.This update captures both the official binary switch and wrapper auth fallback behavior in one place.
docs/MCP_SETUP.md (1)
33-36: Good MCP setup/troubleshooting refresh.The updated guidance is direct and makes the runtime/auth expectations explicit for operators.
Also applies to: 79-82
.cursor/skills/lucky-docker-dev/SKILL.md (1)
3-3: Deploy-preflight and validation guidance looks strong.These additions improve deterministic deploy checks and reduce noisy incident loops.
Also applies to: 6-6, 11-11, 34-59
|
Closed the two actionable CodeRabbit findings on |
4fa9805 to
78c70bc
Compare
|
|
* docs(skills): capture deploy and mcp recovery playbooks * docs(mcp): tighten GitHub wrapper recovery steps
This branch was successfully deployed
1 participant
Summary
github-mcp-serverbinaryValidation
git diff --checkDeploy to Homelabrerun on main (23098473324)codex exec --json --skip-git-repo-check -C /Users/lucassantana/Desenvolvimento/Lucky "Using only the configured github MCP server, list one open issue in repository LucasSantana-Dev/Lucky and return only its number."->243opencode mcp listshowsgithubconnectedopencode run --format json "Using only the configured github MCP server, list one open issue in repository LucasSantana-Dev/Lucky and return only its number."->243Linked
Summary by CodeRabbit