Skip to content

Warn on malformed plugin manifests and log swallowed load errors - #924

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-6812-161-silent-catch-blocks-47-of-all-catches-route-the-high
Sep 12, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-6812-161-silent-catch-blocks-47-of-all-catches-route-the-high

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Warns through the existing plugin warning sink when manifest.json fails to parse or validate, while a missing file stays silent
  • Data-only native manifest.json uses the same parse and read warnings instead of swallowing malformed JSON
  • Metadata-only Claude .claude-plugin/manifest.json is adapted as name and description and is not validated against the corbits schema
  • Logs leftover empty catches in the plugin loader, session index, and TUI runner through existing debug sinks, and comments intentional soft-allows

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • bun run check passes (lint, typecheck, build, guarded test suite)

Fixes CL-6812

@linear-code

linear-code Bot commented Sep 12, 2026

Copy link
Copy Markdown

CL-6812

A broken native manifest was treated as missing, so data-only plugins
inferred kind with no warning. Claude marketplace metadata is name
and description, not a corbits schema, so validating it produced
false id and kind warnings on untrusted loads.
@TheGreatAxios
TheGreatAxios force-pushed the cl-6812-161-silent-catch-blocks-47-of-all-catches-route-the-high branch from 2a85169 to 22472ae Compare September 12, 2026 03:27
@TheGreatAxios
TheGreatAxios merged commit 9f4de35 into main Sep 12, 2026
9 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-6812-161-silent-catch-blocks-47-of-all-catches-route-the-high branch September 12, 2026 03:34
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.

1 participant