add cursor command for deprecating a contract#1591
Conversation
WalkthroughThis PR adds a new contract deprecation command documentation file and updates the existing network deprecation command, while extending the pre-commit hook's secret scanning exclusion list to include the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.cursor/commands/deprecate-network.md (1)
39-47: Add language identifiers to fenced code blocks (MD040).Fenced blocks at Line 39–47 and Line 255+ have no language tag. Suggest
bashfor command examples andtextfor output blocks.♻️ Suggested fix
-``` +```bash /deprecate-network fantom```diff -``` +```bash /deprecate-network fantom harmony evmos```diff -``` +```text Deprecating networks: fantom, harmonyAlso applies to: 255-334
🤖 Fix all issues with AI agents
In @.cursor/commands/deprecate-network.md:
- Around line 112-116: Replace the bare Google Sheets URL in the
deprecate-network markdown bullets with proper Markdown link syntax: locate the
bullet lines that contain the plain URL referencing the Product Target State
spreadsheet (the lines describing "The spreadsheet tracks contract deployments
across networks" and the deprecated networks instructions) and convert the raw
URL into a markdown link like [Product Target State
spreadsheet](https://docs.google.com/...) so the link is not a bare URL; apply
the same change for the other occurrence mentioned in the comment.
| - Display a prominent reminder to manually update the Product Target State spreadsheet | ||
| - The spreadsheet tracks contract deployments across networks: https://docs.google.com/spreadsheets/d/1jX1wfFkSn1s19I_KzMA7vB1kfgGxXUv7kRqwUGJJLF4/edit#gid=0 | ||
| - For deprecated networks: Move the network row(s) to the deprecated section | ||
| - This is a manual step that must be done separately as the spreadsheet is not part of the codebase | ||
|
|
There was a problem hiding this comment.
Convert bare URLs to markdown links (MD034).
Bare URLs in prose (Line 112–115 and Line 225–228) trigger markdownlint. Use markdown link syntax instead.
♻️ Suggested fix
- - The spreadsheet tracks contract deployments across networks: https://docs.google.com/spreadsheets/d/1jX1wfFkSn1s19I_KzMA7vB1kfgGxXUv7kRqwUGJJLF4/edit#gid=0
+ - The spreadsheet tracks contract deployments across networks: [Product Target State spreadsheet](https://docs.google.com/spreadsheets/d/1jX1wfFkSn1s19I_KzMA7vB1kfgGxXUv7kRqwUGJJLF4/edit#gid=0)-- **Spreadsheet URL**: https://docs.google.com/spreadsheets/d/1jX1wfFkSn1s19I_KzMA7vB1kfgGxXUv7kRqwUGJJLF4/edit#gid=0
+- **Spreadsheet URL**: [Product Target State spreadsheet](https://docs.google.com/spreadsheets/d/1jX1wfFkSn1s19I_KzMA7vB1kfgGxXUv7kRqwUGJJLF4/edit#gid=0)Also applies to: 223-229
🤖 Prompt for AI Agents
In @.cursor/commands/deprecate-network.md around lines 112 - 116, Replace the
bare Google Sheets URL in the deprecate-network markdown bullets with proper
Markdown link syntax: locate the bullet lines that contain the plain URL
referencing the Product Target State spreadsheet (the lines describing "The
spreadsheet tracks contract deployments across networks" and the deprecated
networks instructions) and convert the raw URL into a markdown link like
[Product Target State spreadsheet](https://docs.google.com/...) so the link is
not a bare URL; apply the same change for the other occurrence mentioned in the
comment.
Which Jira task belongs to this PR?
Why did I implement it this way?
Checklist before requesting a review
Checklist for reviewer (DO NOT DEPLOY and contracts BEFORE CHECKING THIS!!!)