Skip to content

Revert "fix(skill): 使用 gray-matter 包解析 md 的元信息"#330

Merged
zzcr merged 1 commit intodevfrom
revert-328-shen/skill-just-bash
Feb 14, 2026
Merged

Revert "fix(skill): 使用 gray-matter 包解析 md 的元信息"#330
zzcr merged 1 commit intodevfrom
revert-328-shen/skill-just-bash

Conversation

@zzcr
Copy link
Member

@zzcr zzcr commented Feb 14, 2026

Reverts #328

Summary by CodeRabbit

  • Documentation

    • Enhanced table formatting in technical guides for improved readability and visual alignment.
  • Chores

    • Removed unused internal dependency to optimize bundle size.
    • Refactored skill metadata extraction with redesigned parsing logic for improved maintainability.
    • Updated code structure and formatting across multiple files for consistency.

@zzcr zzcr merged commit ff461ce into dev Feb 14, 2026
2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request removes the gray-matter-browser dependency and replaces it with inline YAML front matter parsing using a regex pattern. The execute function signature in skill tools is updated to accept an args object. Formatting changes are applied to documentation and TypeScript files.

Changes

Cohort / File(s) Summary
Documentation Formatting
docs/guide/tiny-robot-remoter.md
Markdown table reformatted for alignment and readability with no content changes.
Dependency Management
packages/next-sdk/package.json
Removed transitive dependency gray-matter-browser version 4.0.4.
Front Matter Parsing & Skill Tools
packages/next-sdk/skills/index.ts
Introduced FRONT_MATTER_BLOCK_REG regex to extract YAML front matter blocks. Replaced gray-matter-browser parsing with inline logic extracting name and description fields. Updated getSkillOverviews to fetch content by path before parsing. Changed createSkillTools execute function signature from destructured parameters to args object with internal extraction.
Code Formatting
packages/next-remoter/src/composable/useSkill.ts
Reformatted assignment to customAgentProvider.systemPrompt inside watchEffect with no behavioral changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

A rabbit hops through code with glee,
No more gray matter here to see! 🐰
With regex gleaming, sharp and bright,
Front matter parsing—done just right! ✨
Dependencies trimmed, functions refine,
This pull request? Chef's kiss fine! 🥕

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-328-shen/skill-just-bash

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 and usage tips.

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

Comments