agent-skills

0 Fork 0

mirror of https://github.com/addyosmani/agent-skills.git synced 2026-10-26 23:15:04 +00:00

Files

T

Abhishek Sharma 30d4493755 fix(validator): flag the indicators both parsers actually reject

Addresses review on #603.

The comma claim was wrong. The code comment said a leading `,` parses cleanly;
PyYAML rejects it with a ParserError and psych with a SyntaxError. The same
comment was wrong about `]` and `}`. I had asserted all three without measuring,
and a test pinned the comma claim, so the mistake was defended rather than
caught.

Now flagged, each verified against PyYAML and psych: `,` `*` and a one-line
`|`/`>` header. A bare `|` with indented lines under it is a real block scalar
and stays accepted; `|foo` is not one.

Still not flagged, because each parses: `&anchor`, `#`, `:`/`-` without a
following space, and `[`/`{`, which open valid flow collections. `]` and `}` are
rejected by both parsers but left out for now - no frontmatter has ever started
with them, and widening past what was asked risks false positives I have not
thought through.

Docstring no longer says "the three shapes".

2026-09-29 11:47:47 -07:00

lib

fix(validator): flag the indicators both parsers actually reject

2026-09-29 11:47:47 -07:00

run-evals-test.js

refactor(evals): extract clearGradingSlot and test the production path

2026-09-21 02:52:39 +08:00

run-evals.js

refactor(evals): extract clearGradingSlot and test the production path

2026-09-21 02:52:39 +08:00

validate-artifact-paths-test.js

ci: add validator guarding spec/plan/todo artifact paths