Content
85%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.
Highly actionable and clearly sequenced content with explicit validation and destructive-operation gating, held back by a monolithic single-file structure with no reference bundle and some repeated lists. Splitting the acceptance, runtime-setup, and documentation-record material into reference files would lift both structure and conciseness.
Suggestions
Move the message component list (Source, Plain, At, AtAll, Image, Voice, File, Quote, Face, Forward, Unknown) into a single reference file and cite it once, instead of repeating the full enumeration in Unit Tests, Conversion Contract, and Required EBA Acceptance Coverage.
Extract the standalone runtime setup (terminal commands, env vars, probe sweeps) and the documentation record format into references/ files (e.g. references/runtime-setup.md, references/adapter-doc-format.md), keeping SKILL.md as a workflow overview.
Consolidate the evidence-level taxonomy (plugin-e2e-ui, plugin-e2e-protocol, plugin-e2e-outbound, adapter-live, unit, not-supported, blocked) into one section — it is currently defined in Live Test Workflow and re-explained across the acceptance sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and project-specific with no generic concept explanations, but the message component list (Source, Plain, At, AtAll…) is repeated in three sections and the evidence-level taxonomy spans two sections. Not 5: those repetitions and overlaps are tokens that do not earn their place. | 4 / 5 |
Actionability | Provides copy-paste-ready commands (pytest invocations, the three-terminal standalone runtime block with ports and env vars), exact file paths, a directory layout, and an executable conversion-contract code block. Not 4: the commands are complete and cover the common cases, with only appropriate <platform> placeholders. | 5 / 5 |
Workflow Clarity | A numbered 'Start Here' sequence, implementation and test checklists, destructive APIs gated behind flags and "run them last", and a 'Before Finishing' validation list (run tests, git diff --check, stop runtimes, confirm no secrets staged). Not 4: validation checkpoints for the destructive/live operations are explicit and complete, satisfying the cap exception. | 5 / 5 |
Progressive Disclosure | The file is well-sectioned with clear headers and clear pointers into LangBot repo docs, but there is no references/scripts/assets bundle at all — ~300 lines including acceptance-coverage lists, the docs-record format, and repeated component tables are inlined in a single file. Not 4: content that should live in reference files is inline; not 2: structure and external signposting are good. | 3 / 5 |
Total | 17 / 20 Passed |