feat: add confluence-cli skill — Confluence wiki from the terminal #7
No reviewers
Labels
No labels
community-feedback
enhancement
skill-upgrade
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
magnus/agent-skills!7
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feat/confluence-cli-skill"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
CLI wrapper for the Atlassian Confluence Cloud REST API. List spaces, browse pages, view content with body extraction, search with CQL, and create pages.
Files
Commands
me— current user profilespaces— list spaces with keys and IDspages— list pages by space with version infoview— full page content with HTML body extraction (stripped to plain text)search— CQL search with result excerpts and highlighted snippetscreate— create pages with HTML body and parent page supportFeatures
Setup
Token is free from id.atlassian.com/manage/api-tokens. Same token works for jira-cli.
QA Gate (10/10 tests passing)
Signed-off-by: Jasper magnus@groktop.us
First-Pass Code Review: confluence-cli skill
Automated review by Jasper. 4 issues found.
1.
import reinside function body (line 339)import reis called insidecmd_view()rather than at module level. Python caches imports, so this works, but static-analysis tools and linters expect imports at the top of the file.Fix: Move
import reto line 10 (with the other stdlib imports).2.
--forceflag declared but never implemented (lines 54, 60)The
--forceglobal flag is parsed and stored, but no command handler ever reads it. Users who try--forcewill get a silent no-op. Either implement a behavior for it (force-create, skip confirmation) or remove it from the flag definitions.3. Error message suggests non-existent
--serverflag (line 120)The connection error message says
Check CONFLUENCE_SERVER or use --server, but there is no--serverCLI flag anywhere in the parser. The server is only configurable viaCONFLUENCE_SERVERenv var. Fix: dropor use --serveror add the flag.4.
cmd_pagessilently falls back when space not found (lines 268-272)When
--space BADSPACEdoesn't resolve,space_idstaysNoneandlist_pages()returns pages from all spaces without warning the user. Compare withcmd_create(lines 426-431) which correctly callsdie()on the same condition.cmd_pagesshould also error out.Overall the CLI is well-structured and follows cli-builder conventions. These are all minor issues — none are blockers.
Inline code comments (supplemental to previous review).
Style:
import reinside function body. Move to module-level imports.--forcedeclared but never used by any command handler.Error message references non-existent
--serverflag.