Skip to content

fix(analyzer): project manifest version for RP3 - #474

Open
agentsope wants to merge 1 commit into
NVIDIA:mainfrom
agentsope:fix/rp3-manifest-version
Open

fix(analyzer): project manifest version for RP3#474
agentsope wants to merge 1 commit into
NVIDIA:mainfrom
agentsope:fix/rp3-manifest-version

Conversation

@agentsope

Copy link
Copy Markdown

Summary

  • project the version field from SKILL.md frontmatter into the bounded manifest
  • add a regression test that exercises real frontmatter through build_context before running the RP3 analyzer
  • update the manifest parser docstring to include the projected field

Before this change, RP3 ran successfully but could not observe version, so values such as "*" produced no finding.

Fixes #472.

Testing

  • PATH="$PWD/.venv/bin:$PATH" make test-unit (3979 passed, 14 skipped, 38 deselected, 4 xfailed)
  • PATH="$PWD/.venv/bin:$PATH" make lint format-check
  • manual no-LLM CLI repro: version: "*" now emits RP3

Signed-off-by: Whj9283 <1621370123@qq.com>
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.

RP3 gates on a manifest version key that build_context never populates, so the rule never fires

1 participant