Conversation
yordis
commented
May 26, 2026
yordis
commented
Member
- Product and feature authors need a shared structure so engineers, reviewers, and AI agents can pick up requirements work without losing intent or duplicating context.
- A dedicated plugin keeps PRD and FRD authoring decoupled from blueprints and work orders, matching the rest of the TrogonStack skill catalog.
Authors and reviewers need consistent structure for product-level context and feature-level requirements so engineering can pick up work without losing intent. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryLow Risk Overview The plugin introduces agent skills that scaffold and author requirements under Product overview (PRD): six default section skills plus Feature requirements (FRD): A README documents the operating model, file layout, and install commands for the marketplace. Reviewed by Cursor Bugbot for commit 5f85e23. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Warning Review limit reached
More reviews will be available in 38 minutes and 50 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 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 configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (14)
WalkthroughThis PR introduces ChangesTrogonStack Product Requirements Plugin
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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: 15
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@plugins/trogonstack-product-requirements/.claude-plugin/plugin.json`:
- Line 3: Update the "description" value in plugin.json so it mentions both
Product Requirements Documents (PRDs) and Functional Requirements Documents
(FRDs) to match the README/marketplace; locate the "description" property in
plugins/trogonstack-product-requirements/.claude-plugin/plugin.json and change
its text from only referencing PRDs to something inclusive like "Skills for
writing and reviewing Product Requirements Documents (PRDs) and Functional
Requirements Documents (FRDs)" ensuring wording matches other metadata entries.
In `@plugins/trogonstack-product-requirements/skills/frd-split/SKILL.md`:
- Around line 82-88: The fenced code block showing the directory tree (the block
containing ".trogonai/project/{projectid}/frd/{parent-slug}/", "index.frd.md",
"{child-1-slug}.frd.md", etc.) lacks a language specifier and triggers MD040;
fix it by adding a language label (e.g., text) after the opening triple
backticks in SKILL.md so the block begins with ```text and leave the tree
contents unchanged.
In `@plugins/trogonstack-product-requirements/skills/frd-write/SKILL.md`:
- Around line 160-181: The markdown fenced code blocks in SKILL.md around the
REQ-CHK-003 examples are unlabeled and trigger markdownlint MD040; update both
fenced blocks that contain the "REQ-CHK-003: Payment Confirmation" good and "Bad
(reject this shape)" examples to include a language identifier (e.g.,
```markdown) at the opening backticks so the blocks become ```markdown ... ```
and ensure the closing backticks remain unchanged; search for the REQ-CHK-003
code examples in the file to locate the exact blocks to modify.
In
`@plugins/trogonstack-product-requirements/skills/prd-business-problem/SKILL.md`:
- Line 93: Several fenced code blocks in SKILL.md use plain ``` markers without
a language tag (e.g., the examples that start with the "## The Problem"
heading), which triggers MD040; update each triple-backtick fence to include a
language identifier (use "markdown" for these prose examples) so they become
```markdown ... ```, and apply the same change to the other affected fence near
the second "## The Problem" instance; search for plain ``` fences around those
headings and replace them with ```markdown to satisfy markdownlint and keep
tooling consistent.
In
`@plugins/trogonstack-product-requirements/skills/prd-current-state/assets/current-state-template.md`:
- Around line 17-20: Add a blank line between the "## Cost of the status quo"
heading and the table to satisfy MD058; locate the heading text "## Cost of the
status quo" and insert one empty line before the table rows starting with "|
Cost | Measure | Source |".
In `@plugins/trogonstack-product-requirements/skills/prd-current-state/SKILL.md`:
- Line 83: Two fenced code blocks around the "How the job gets done today"
examples are missing a language tag (triggers markdownlint MD040); update each
opening triple-backtick fence to include a language (e.g., change ``` to
```markdown) for both example blocks so the fences read ```markdown, keeping the
existing block content unchanged and preserving both occurrences of the "How the
job gets done today" example sections.
In
`@plugins/trogonstack-product-requirements/skills/prd-custom-overview/SKILL.md`:
- Around line 100-121: The two fenced example blocks in SKILL.md (the one
starting with the "Rollout Strategy" heading and the "Misc Notes" block) are
missing language identifiers which triggers MD040; update both opening
triple-backtick fences to include the markdown language tag (```markdown) so the
linter passes and the examples are explicitly marked as Markdown.
In `@plugins/trogonstack-product-requirements/skills/prd-personas/SKILL.md`:
- Line 90: The fenced code blocks in SKILL.md around the headings "## Primary:
Mid-market Account Executive" and "## Primary: Sales user" lack language
identifiers and trigger markdownlint MD040; update the opening triple-backtick
fences for those two examples to include "markdown" (i.e., change ``` to
```markdown) so both example fences are annotated and the linter warning is
resolved.
In
`@plugins/trogonstack-product-requirements/skills/prd-product-description/SKILL.md`:
- Line 100: Two fenced example blocks around the "One-liner" headings are
missing language identifiers and trigger markdownlint MD040; edit the two fenced
code blocks that contain the "## One-liner" examples and change their opening
fences from ``` to ```markdown so both example blocks explicitly declare the
markdown language (apply this to both occurrences noted in the file).
- Line 57: The markdown heading "Boundaries (what the product is NOT**" has
unbalanced emphasis and a missing closing parenthesis; update the text in
SKILL.md so the bold and parentheses are balanced, e.g., change the fragment to
"Boundaries (what the product is NOT)" and wrap the whole label in bold as
"**Boundaries (what the product is NOT)**" so the emphasis and parentheses are
correctly closed.
In `@plugins/trogonstack-product-requirements/skills/prd-review/SKILL.md`:
- Around line 38-46: The fenced filesystem tree in SKILL.md is missing a
language identifier, causing MD040; update the opening fence for the snippet
that shows ".trogonai/project/{projectid}/prd/" to use a language specifier (use
text) so the block becomes "```text" instead of "```" and keep the closing fence
unchanged; locate the snippet in SKILL.md and add the word "text" immediately
after the opening triple backticks.
In
`@plugins/trogonstack-product-requirements/skills/prd-success-metrics/assets/success-metrics-template.md`:
- Around line 6-20: The markdown violates MD058 because the tables are not
surrounded by blank lines; update each section (headers "## Primary outcome
metric", "## Leading indicators", and "## Guardrail metrics (must not regress)")
by inserting a blank line immediately after the heading and another blank line
after the table block so that each table is preceded and followed by an empty
line.
In
`@plugins/trogonstack-product-requirements/skills/prd-success-metrics/SKILL.md`:
- Around line 105-148: The MD040 lint is triggered because the fenced code
blocks in SKILL.md lack a language tag; edit the example fences around the
primary outcome/leading/guardrail/decision rules examples and add "md" to each
opening triple-backtick (i.e., change ``` to ```md) so the examples are
explicitly marked as Markdown and the linter no longer flags MD040.
In
`@plugins/trogonstack-product-requirements/skills/prd-technical-requirements/assets/technical-requirements-template.md`:
- Around line 6-37: The markdown linter MD058 complains because the tables under
the "## Performance" and "## Integrations" sections need surrounding blank
lines; modify the template so there is an empty line before and after each table
block (the Performance table starting under "## Performance" and the
Integrations table under "## Integrations") to satisfy MD058 while preserving
the header and table rows.
In
`@plugins/trogonstack-product-requirements/skills/prd-technical-requirements/SKILL.md`:
- Around line 145-188: Add the language tag "md" to both fenced-code blocks that
start with the "## Performance" headings (the detailed example block and the
"Bad" example block) so they read as ```md ... ```; update the opening backticks
for the first large example and the second "Bad" example to include md to
satisfy the MD040 lint rule and ensure proper rendering.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 1fe25eee-8570-4849-8204-2d580dfed1c9
📒 Files selected for processing (27)
.claude-plugin/marketplace.json.github/release-please-config.json.github/release-please-manifest.jsonplugins/trogonstack-product-requirements/.claude-plugin/plugin.jsonplugins/trogonstack-product-requirements/README.mdplugins/trogonstack-product-requirements/skills/frd-getting-started/SKILL.mdplugins/trogonstack-product-requirements/skills/frd-review/SKILL.mdplugins/trogonstack-product-requirements/skills/frd-split/SKILL.mdplugins/trogonstack-product-requirements/skills/frd-write/SKILL.mdplugins/trogonstack-product-requirements/skills/frd-write/assets/frd-template.mdplugins/trogonstack-product-requirements/skills/prd-business-problem/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-business-problem/assets/business-problem-template.mdplugins/trogonstack-product-requirements/skills/prd-current-state/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-current-state/assets/current-state-template.mdplugins/trogonstack-product-requirements/skills/prd-custom-overview/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-custom-overview/assets/custom-overview-template.mdplugins/trogonstack-product-requirements/skills/prd-getting-started/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-personas/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-personas/assets/personas-template.mdplugins/trogonstack-product-requirements/skills/prd-product-description/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-product-description/assets/product-description-template.mdplugins/trogonstack-product-requirements/skills/prd-review/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-success-metrics/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-success-metrics/assets/success-metrics-template.mdplugins/trogonstack-product-requirements/skills/prd-technical-requirements/SKILL.mdplugins/trogonstack-product-requirements/skills/prd-technical-requirements/assets/technical-requirements-template.mdplugins/trogonstack-product-requirements/skills/requirements-operating-model/SKILL.md
…larify scope Lint violations made the plugin look untrustworthy to anyone running standard markdown tooling, and the plugin description omitted FRDs even though the catalog markets both PRD and FRD authoring. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
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
1 participant
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.