Skip to content

fix: clean YAML frontmatter output#71

Merged
gotalab merged 1 commit into
mainfrom
fix/frontmatter-and-bump-1-1-0
Dec 29, 2025
Merged

fix: clean YAML frontmatter output#71
gotalab merged 1 commit into
mainfrom
fix/frontmatter-and-bump-1-1-0

Conversation

@gotalab

@gotalab gotalab commented Dec 29, 2025

Copy link
Copy Markdown
Owner

Description & Motivation

skillport meta set was rewriting YAML frontmatter into JSON-like quoted keys (e.g., "name":) and wrapping long strings using \\ line continuations.
This PR fixes the frontmatter writer so SKILL.md remains clean, readable YAML.

Also bumps non-root package versions to 1.1.0 (next planned version).

Changes

  • Fix frontmatter serialization to avoid quoted keys and \\ continuations
  • Add unit test coverage for frontmatter formatting
  • Bump packages/skillport-core and packages/skillport-mcp versions to 1.1.0 and update pinned deps

How to Test

uv run ruff check .
uv run pytest -q
SKILLPORT_SKILLS_DIR=.skills SKILLPORT_EMBEDDING_PROVIDER=none uv run verify_server.py

Checklist

  • Lint passes
  • Tests pass
  • Docs updated (if behavior changed)

Copilot AI review requested due to automatic review settings December 29, 2025 16:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes YAML frontmatter output formatting and prepares for a 1.1.0 release by bumping package versions and updating path conventions.

Key Changes:

  • Implements clean YAML frontmatter output without JSON-like quoted keys or line wrapping
  • Adds test coverage for frontmatter formatting
  • Bumps skillport-core and skillport-mcp to version 1.1.0
  • Standardizes skills directory path from .agent/skills to .skills
  • Adds new experimental skills (skill-evaluator, opus-4-5-migration, git-branch-cleanup)
  • Removes deprecated .agent/skills content

Reviewed changes

Copilot reviewed 91 out of 102 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/skillport-core/src/skillport/interfaces/cli/commands/meta.py Core YAML frontmatter formatting fix with intelligent string quoting
tests/unit/test_meta_frontmatter_format.py New test preventing regression of YAML formatting issues
packages/skillport-core/pyproject.toml Version bump to 1.1.0
packages/skillport-mcp/pyproject.toml Version bump to 1.1.0 and dependency update
pyproject.toml Root package version and workspace configuration updates
uv.lock Lock file updates for version changes and workspace members
tests/unit/*.py Test file updates for new .skills path convention
guide/*.md Documentation updates for new path convention
.skills/experimental/* New experimental skills added (evaluator, migration, cleanup)
.agent/skills/* Deprecated skill files removed

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gotalab gotalab force-pushed the fix/frontmatter-and-bump-1-1-0 branch from 5eb6f62 to 1cf7823 Compare December 29, 2025 16:58
@gotalab gotalab force-pushed the fix/frontmatter-and-bump-1-1-0 branch from 1cf7823 to 5012221 Compare December 29, 2025 17:00
@gotalab gotalab merged commit 40ccca6 into main Dec 29, 2025
3 checks passed
@gotalab gotalab deleted the fix/frontmatter-and-bump-1-1-0 branch December 29, 2025 17:03
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.

2 participants