Skip to content

docs: add v0.1.3 changelog and skip CI for docs-only PRs#9

Merged
simongdavies merged 1 commit into
hyperlight-dev:mainfrom
simongdavies:release-v0.1.3
Mar 24, 2026
Merged

docs: add v0.1.3 changelog and skip CI for docs-only PRs#9
simongdavies merged 1 commit into
hyperlight-dev:mainfrom
simongdavies:release-v0.1.3

Conversation

@simongdavies
Copy link
Copy Markdown
Member

@simongdavies simongdavies commented Mar 24, 2026 •
edited
Loading

  • Add v0.1.3 changelog entry for plugin .ts/.js loading fix
  • Skip lint, build, and test jobs when only docs change
  • Uses dorny/paths-filter to detect code vs docs-only changes

Copilot AI review requested due to automatic review settings March 24, 2026 08:28
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a v0.1.3 changelog entry and updates the PR validation workflow to skip the heavier CI jobs when changes don’t touch “code” paths (via dorny/paths-filter), while keeping a single status gate job.

Changes:

  • Added v0.1.3 release notes to the changelog.
  • Introduced a changes job in PR validation to detect code vs non-code changes.
  • Conditioned lint/test/build/docker jobs on the paths filter result.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
CHANGELOG.md Adds v0.1.3 “Fixed” entries and the release link reference.
.github/workflows/pr-validate.yml Adds a paths-filter-based “Detect Changes” job and gates expensive CI jobs on its output.

Comment thread .github/workflows/pr-validate.yml Outdated
@simongdavies simongdavies force-pushed the release-v0.1.3 branch 2 times, most recently from 10051dc to 100317e Compare March 24, 2026 08:38
@simongdavies simongdavies requested a review from Copilot March 24, 2026 08:40
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/pr-validate.yml
Comment thread .github/workflows/pr-validate.yml Outdated
@simongdavies simongdavies requested a review from Copilot March 24, 2026 08:48
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/pr-validate.yml Outdated
@simongdavies simongdavies requested a review from Copilot March 24, 2026 09:06
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/pr-validate.yml Outdated
@simongdavies simongdavies requested a review from Copilot March 24, 2026 09:09
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/pr-validate.yml
Comment thread .github/workflows/pr-validate.yml
@simongdavies
docs: add v0.1.3 changelog and skip CI for docs-only PRs
024f85d 
- Add v0.1.3 changelog entry for plugin .ts/.js loading fix
- Skip lint, build, and test jobs when only markdown files change
- Uses dorny/paths-filter with '**' + '!**/*.md' pattern
- Gate ci-status on changes job to catch filter failures

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
@simongdavies simongdavies requested a review from Copilot March 24, 2026 09:22
Copilot AI reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@simongdavies simongdavies merged commit 2ca15c8 into hyperlight-dev:main Mar 24, 2026
13 checks passed
@simongdavies simongdavies mentioned this pull request May 14, 2026
Merged
simongdavies added a commit to simongdavies/hyperagent that referenced this pull request May 14, 2026
@simongdavies
fix: address PR hyperlight-dev#137 review feedback (9 issues)
15c663f 
- Fix hyperlight-dev#2: Use MCP_SETUP_COMMANDS lookup for correct CLI setup hints
  (was generating 'setup-rti-mcp' instead of '--mcp-setup-fabric-rti')
- Fix hyperlight-dev#3: Show unconfigured status when mcpManager is null
- Fix hyperlight-dev#4: Update test assertion to match corrected setup command
- Fix hyperlight-dev#5: Add highlight.js ^10.7.3 as explicit dependency (was transitive)
- Fix hyperlight-dev#6: Validate Fabric RTI options — reject missing values & unknown flags
- Fix hyperlight-dev#7: Set HYPERAGENT_VERBOSE=1 from cli.verbose flag
- Fix hyperlight-dev#8: Update skill count from 9 to 10 in docs/SKILLS.md
- Fix hyperlight-dev#9: Use stderr 'ignore' instead of 'pipe' to avoid back-pressure
- Fix hyperlight-dev#10: Add CLI parser test for --mcp-setup-fabric-rti flag

Comment hyperlight-dev#1 (hljs types) is invalid — typecheck passes with /// <reference>.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
simongdavies added a commit that referenced this pull request May 14, 2026
@simongdavies
fix: address remaining PR #135 review feedback (#136)
17134a1 
- /markdown toggle: set sessionNeedsRebuild so system prompt updates (#1)
- CLI help: document --[no-]markdown, --md/--no-md aliases, HYPERAGENT_MARKDOWN (#6)
- Streamed output: gate renderMarkdown on looksLikeMarkdown consistently (#7)
- markdown-renderer: use local Marked instance instead of global setOptions (#9)
- looksLikeMarkdown: remove over-eager bold and unordered-list patterns (#10)
- unescape: verified valid marked-terminal option (comment was wrong) (#8)
- linkifyFiles order: verified safe — [[file:]] not a markdown token (#16)

Verified: diff matches this message. 40 test files, 2350 tests pass.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
simongdavies added a commit that referenced this pull request May 14, 2026
@simongdavies
Feat kql expert (#137)
b646352 
* feat: KQL expert skill with requires-mcp, trigger cleanup, syntax highlighting

- Add kql-expert skill (SKILL.md) with 35 triggers, requires-mcp: fabric-rti-mcp
- Add requires-mcp frontmatter support in skill-loader + approach-resolver
- Add MCP enrichment wiring in agent index.ts
- Add --mcp setup-fabric-rti CLI command
- Clean generic triggers across 7 skills to prevent false matches
- Add hljs-kql.ts grammar (derived from @kusto/monaco-kusto MIT)
- Register KQL/Kusto syntax highlighting in markdown-renderer via createRequire
- Add intent-matcher regression tests (27 tests: 22 positive + 5 guards)
- Add skill-loader tests (8 tests) and approach-resolver MCP tests (8 tests)

All 2396 tests pass. Typecheck clean. Fmt clean.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

* fix: gate diagnostic output behind --verbose/--debug flags

- Rust: gate [hyperlight-analysis] eprintln! behind HYPERAGENT_VERBOSE/DEBUG env vars
- MCP: gate [mcp] Connected/Auth success messages behind isVerbose()
- MCP: pipe subprocess stderr when not verbose to suppress Python INFO logs
- KQL skill: remove azuremcpserver reference, add anti-pattern to use fabric-rti-mcp only

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

* docs: update skill tables and add requires-mcp documentation

- Add all 9 skills to tables in docs/SKILLS.md, README.md, skills/CLAUDE.md,
  and .github/instructions/skills.instructions.md (was missing kql-expert,
  xlsx-expert, mcp-services in various files)
- Document requires-mcp frontmatter field in docs/SKILLS.md
- Add requires-mcp to YAML examples in instruction files
- Alphabetise skill tables for consistency

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

* fix: address PR #137 review feedback (9 issues)

- Fix #2: Use MCP_SETUP_COMMANDS lookup for correct CLI setup hints
  (was generating 'setup-rti-mcp' instead of '--mcp-setup-fabric-rti')
- Fix #3: Show unconfigured status when mcpManager is null
- Fix #4: Update test assertion to match corrected setup command
- Fix #5: Add highlight.js ^10.7.3 as explicit dependency (was transitive)
- Fix #6: Validate Fabric RTI options — reject missing values & unknown flags
- Fix #7: Set HYPERAGENT_VERBOSE=1 from cli.verbose flag
- Fix #8: Update skill count from 9 to 10 in docs/SKILLS.md
- Fix #9: Use stderr 'ignore' instead of 'pipe' to avoid back-pressure
- Fix #10: Add CLI parser test for --mcp-setup-fabric-rti flag

Comment #1 (hljs types) is invalid — typecheck passes with /// <reference>.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

---------

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@simongdavies