Skip to content

fix(store): drop banned 'Obsidian' from manifest description; remove README placeholder#71

Merged
vreshch merged 1 commit into
masterfrom
fix/desc-obsidian
Jun 21, 2026
Merged

fix(store): drop banned 'Obsidian' from manifest description; remove README placeholder#71
vreshch merged 1 commit into
masterfrom
fix/desc-obsidian

Conversation

@vreshch

@vreshch vreshch commented Jun 21, 2026

Copy link
Copy Markdown
Member

Second pre-submission checker pass surfaced one ERROR (hard blocker) and one Warning. (The build-provenance attestations from 0.2.2 now Pass for both main.js and styles.css.)

Fixed

  • ERROR - manifest description contained "Obsidian". The store bans the word in the description (redundant inside the plugin directory). PR feat: MCP-led listing - network icon + new store description (0.2.1) #69 had introduced "...read and write your Obsidian vault...". Changed to "...read and write your vault...". Now 195 chars, ends with a period, no emoji.
  • Warning - README placeholder text. The Community-plugins install section read "*Coming soon - pending Obsidian review.*" - "Coming soon" trips the checker's unfinished-placeholder detector. Reworded to "Once it is listed in the directory: ...".

Note

The community-plugins.json submission entry must mirror the manifest description byte-for-byte, so the fork entry is being updated to the new text alongside this.

Verify

npm run verify green locally (68 tests + type-check/lint/format/build/check:hosts/docs/bundle). Release 0.2.3.

…README placeholder

- manifest.json: store rejects the word 'Obsidian' in the description (redundant in the plugin directory). 'your Obsidian vault' -> 'your vault'. 195 chars, ends with a period, no emoji.

- README: the install section's '*Coming soon - pending Obsidian review.*' tripped the checker's placeholder-text detector; reworded to 'Once it is listed in the directory:'.

Release 0.2.3.
@github-actions

Copy link
Copy Markdown

🎉 PR Validation ✅ PASSED

Commit: b62b82f0a284bbad6230e420104cfb4ddf875e29
Branch: fix/desc-obsidian

Checks:

  • ✅ Dependencies installed
  • ✅ Type check passed
  • ✅ Linting passed
  • ✅ Format check passed
  • ✅ Tests + coverage passed
  • ✅ Build successful
  • ✅ Store-compliance checks passed
  • ✅ Bundle is mobile-safe

Ready to merge!


🔗 View workflow run

@vreshch vreshch merged commit 0940765 into master Jun 21, 2026
1 check passed
@vreshch vreshch deleted the fix/desc-obsidian branch June 21, 2026 12:07
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