Skip to content

CC-177: Add <MarkdownRenderer/> to Component Library - #33

Merged
ianpaschal merged 1 commit into
mainfrom
ian/cc-177-add-markdownrenderer-to-component-library
Sep 11, 2026
Merged

ianpaschal merged 1 commit into
mainfrom
ian/cc-177-add-markdownrenderer-to-component-library

Conversation

@ianpaschal

Copy link
Copy Markdown
Owner

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 1bc2d429-dca7-48c2-9fd6-96ad0ca40d49

📥 Commits

Reviewing files that changed from the base of the PR and between 803ede6 and 6c47fdf.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (7)
  • package.json
  • src/components/MarkdownRenderer/MarkdownRenderer.module.scss
  • src/components/MarkdownRenderer/MarkdownRenderer.module.scss.d.ts
  • src/components/MarkdownRenderer/MarkdownRenderer.stories.tsx
  • src/components/MarkdownRenderer/MarkdownRenderer.tsx
  • src/components/MarkdownRenderer/index.ts
  • src/index.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

Adds a typed MarkdownRenderer component that sanitizes Markdown with DOMPurify and renders it with ReactMarkdown. Adds component styling, SCSS type declarations, and a Storybook story. Adds the required dependencies and exports the component and its props through the component and package entry points.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 6c47f

No concrete merge-blocking risk remains in the Markdown renderer changes.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so the description cannot be evaluated for relevance to the changeset. Add a brief description that explains the new MarkdownRenderer component, its sanitization behavior, styling, Storybook story, and public exports.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the primary change: adding the MarkdownRenderer component to the component library.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 5…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

package.json

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

src/components/MarkdownRenderer/MarkdownRenderer.module.scss.d.ts

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).

src/components/MarkdownRenderer/MarkdownRenderer.stories.tsx

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).

  • 3 others
🔧 Stylelint (17.14.0)
src/components/MarkdownRenderer/MarkdownRenderer.module.scss

ConfigurationError: Could not find "stylelint-config-clean-order". Do you need to install the package or use the "configBasedir" option?
at getModulePath (file:///usr/local/lib/node_modules/stylelint/lib/utils/getModulePath.mjs:38:9)
at loadExtendedConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:285:21)
at extendConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:252:25)
at async augmentConfigBasic (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:85:20)
at async augmentConfigFull (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:138:24)
at async getConfigForFile (file:///usr/local/lib/node_modules/stylelint/lib/getConfigForFile.mjs:102:26)
at async resolveOptionValue (file:///usr/local/lib/node_modules/stylelint/lib/utils/resolveOptionValue.mjs:27:24)
at async standalone (file:///usr/local/lib/node_modules/stylelint/lib/standalone.mjs:127:22)


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.

@ianpaschal
ianpaschal merged commit 83258c3 into main Sep 11, 2026
2 checks passed
@ianpaschal
ianpaschal deleted the ian/cc-177-add-markdownrenderer-to-component-library branch September 11, 2026 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant