Skip to content

fix(reqstool-openspec): log non-JSON lines instead of silently swallowing them#27

Merged
jimisola merged 4 commits into
mainfrom
fix/issue-26-openspec-hooks-catch-logging
May 25, 2026
Merged

fix(reqstool-openspec): log non-JSON lines instead of silently swallowing them#27
jimisola merged 4 commits into
mainfrom
fix/issue-26-openspec-hooks-catch-logging

Conversation

@jimisola

@jimisola jimisola commented May 25, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace empty catch {} in McpStdioClient.handle() with console.warn so malformed/non-JSON stdout from reqstool mcp is visible for debugging
  • Bump template version 0.1.00.1.1 in the file header
  • Bump plugin version 0.2.00.2.1
  • Bump marketplace version 0.4.00.4.1
  • Remove redundant .github/plugin/marketplace.json — GitHub Copilot also reads .claude-plugin/marketplace.json, so one file is sufficient
  • Update CLAUDE.md versioning section: document template file header as a third version to bump, and clarify that both Claude Code and Copilot read .claude-plugin/marketplace.json

Test plan

  • Install the updated plugin and confirm the template generated by reqstool-openspec-init contains the console.warn catch block
  • Manually emit a non-JSON line on stdout while the hook is active and verify the warning appears in the console without crashing the client

Closes #26

jimisola added 4 commits May 25, 2026 22:45
…wing them

Replace empty catch{} in McpStdioClient.handle() with a console.warn so
malformed or non-JSON stdout from reqstool mcp is visible for debugging.
Bump template version to 0.1.1, plugin to 0.2.1, marketplace to 0.4.1.

Closes #26

Signed-off-by: Jimisola Laursen <jimisola@jimisola.com>
…s.ts

Signed-off-by: Jimisola Laursen <jimisola@jimisola.com>
…place.json

Signed-off-by: Jimisola Laursen <jimisola@jimisola.com>
Both Claude Code and GitHub Copilot read .claude-plugin/marketplace.json,
so the duplicate under .github/plugin/ is unnecessary. Update CLAUDE.md
to reflect the single source of truth.

Signed-off-by: Jimisola Laursen <jimisola@jimisola.com>
@jimisola jimisola self-assigned this May 25, 2026
@jimisola jimisola merged commit 9aba948 into main May 25, 2026
4 checks passed
@jimisola jimisola deleted the fix/issue-26-openspec-hooks-catch-logging branch May 25, 2026 21:00
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.

reqstool-openspec: empty catch{} in openspecui.hooks.ts silently swallows JSON parse errors

1 participant