Skip to content

fix: correct MCP auth docs and add the missing elnora-review skill#28

Merged
rjamul-elnora-ai merged 1 commit into
mainfrom
fix/readme-auth-and-review-skill
Jul 14, 2026
Merged

fix: correct MCP auth docs and add the missing elnora-review skill#28
rjamul-elnora-ai merged 1 commit into
mainfrom
fix/readme-auth-and-review-skill

Conversation

@rjamul-elnora-ai

Copy link
Copy Markdown
Member

Two fixes

1. The Authentication section was wrong. It said API-key auth means "Pass a Bearer token in the Authorization header" — but the MCP server read API keys only from X-API-Key, so following the docs returned 401 invalid_token. Reworded to show both headers (X-API-Key and Authorization: Bearer — both accepted once the paired server change lands) and when to use each.

2. The elnora-review skill was missing. It shipped in the CLI (v2.5.0, KB review queue) but the release skills-sync didn't copy it (a separate CLI-workflow fix addresses the root cause). Seeded it here so plugin users get it now. Bumped the skill count to 10 and refreshed the elnora-folders description (folders are Knowledge Base folders now, and support sharing).

Pairs with

  • elnora-mcp-server — accept an API key sent as Authorization: Bearer (makes the reworded docs true).
  • elnora-cli — fix the release sync-skills job so new skills sync automatically (root cause of the missing skill).

- Authentication section said API-key auth means "pass a Bearer token in the
  Authorization header", but the server read keys only from X-API-Key — so
  following the docs produced a 401. Reworded to show both headers
  (X-API-Key and Authorization: Bearer now both work) and when to use each.
- Add the elnora-review skill (Knowledge Base review queue), which shipped in
  the CLI but was missed by the skills sync. Update the skill count to 10 and
  refresh the elnora-folders description (folders are Knowledge Base folders now).

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

@ckivisild-elnora-ai ckivisild-elnora-ai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good, reviewed and approved

@rjamul-elnora-ai rjamul-elnora-ai merged commit 7a1e365 into main Jul 14, 2026
5 checks passed
@rjamul-elnora-ai rjamul-elnora-ai deleted the fix/readme-auth-and-review-skill branch July 14, 2026 21:04
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.

2 participants