Content
67%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content is a solid, actionable integration guide: every command is concrete, auth edge cases (headless login, BUILDING/CLIENT_ACTION_REQUIRED states) are handled, and the proxy escape hatch is well documented. Its weaknesses are the padded what-is-Clockify intro and non-informative entity list, plus dangling 'Step 2' references that break the main workflow's coherence.
Suggestions
Delete or shrink the 'Clockify is a time tracking tool used by teams...' intro paragraph and the bare entity bullet list — Claude already knows what Clockify is; keep only entity names that map to actions.
Fix the dangling workflow references: 'skip to **Step 2**' appears twice but no Step 2 exists — either add numbered step headings (1. Connect, 2. Discover actions, 3. Run) or reword to point at the 'Searching for actions' section.
Add one complete end-to-end example, e.g. `membrane action run create-time-entry --connectionId=... --input '{"start":"...","projectId":"..."}' --json` and note how to verify success from the response, to close the actionability and workflow-validation gaps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean commands, but the opening paragraph ("Clockify is a time tracking tool used by teams and individuals to monitor work hours across projects. It helps users track productivity, attendance, and billable hours...") explains what Clockify is — a concept Claude already knows — and the "Clockify Overview" entity list consists mostly of bare nouns ("**Project**", "**Task**", "**Client**") that add no actionable information. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened', below anchor 4 where only minor trimming is needed. | 3 / 5 |
Actionability | Concrete, copy-paste-ready commands appear throughout: `npm install -g @membranehq/cli@latest`, `membrane login --tenant --clientName=<agentType>`, `membrane connection ensure "https://clockify.me/" --json`, `membrane action run <actionId> --connectionId=... --input '{"key": "value"}'`, plus a full proxy flag table and a popular-actions table. It falls short of anchor 5 because there is no complete end-to-end example with a real action ID and input, and the referenced "Step 2" is never defined, leaving a gap in executable guidance. | 4 / 5 |
Workflow Clarity | The sequence — install CLI, authenticate (including headless fallback with `membrane login complete <code>`), `connection ensure`, poll with `--wait` until READY, handle `CLIENT_ACTION_REQUIRED` via clientAction fields, search actions by intent, run, then proxy fallback — is clear with explicit state-handling checkpoints. It does not reach anchor 5 because 'Step 2' is referenced twice but no Step 2 section exists, and running/creating actions has no validation of output (e.g. checking the `output` field or error states). | 4 / 5 |
Progressive Disclosure | The single SKILL.md is well sectioned with headers and tables, and the full action catalog is correctly kept out of the file in favor of runtime discovery via `membrane action list --intent`, which is the right disclosure pattern for a dynamic catalog. It sits at anchor 4 rather than 5 because the 20-row popular-actions table and the entity overview are inline content that could live in a reference file, and there are no bundle files at all to split into. | 4 / 5 |
Total | 15 / 20 Passed |