Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with concrete SDK calls, code, and version pins, and is well-structured with clear sections. Its weak spots are missing validation/feedback loops for risky operations and three referenced task files that do not actually exist in the bundle.
Suggestions
Add explicit validation/retry checkpoints for the auth-credential and connection workflows (e.g., verify the Lakebase-scoped token works before trusting the pool; retry loop for scale-to-zero wake-up) to lift workflow clarity above 2.
Provide the referenced task files (connections.md, operations.md, reverse-etl.md) or remove the 'Task files' pointers, since the bundle currently points to files that do not exist.
Move the inline region lists and version pin block into a bundled reference or a clearly dated/old-patterns section so the main body stays lean and the time-sensitive data is isolated.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and largely free of concept-explanation padding (no 'what is Postgres' filler), but inline time-sensitive facts — a region list, version pins, and the pip install block — add bulk that isn't placed in a deprecated/old-patterns section, keeping it just off the lean 'every token earns its place' level. | 2 / 3 |
Actionability | Provides concrete executable guidance throughout: `generate_database_credential`, `pip install` version pins, a `ConnectionPool`/`OAuthConnection` recipe, `max_lifetime=2700`, `WorkspaceClient().postgres`, `.wait()`, `FieldMask`, `sslmode=require`, and the `postgresql+psycopg://` URL prefix — copy-paste ready. | 3 / 3 |
Workflow Clarity | Steps are clearly sequenced (numbered lead-connection pattern, resource model, create/update flow with `.wait()`/`FieldMask`), but for auth-critical and database operations there are no explicit validate→fix→retry checkpoints or feedback loops, which caps workflow clarity at 2 per the destructive/batch-operations guideline. | 2 / 3 |
Progressive Disclosure | The body has clear sections and a 'Task files' list signaling one-level-deep references to connections.md, operations.md, and reverse-etl.md, but those referenced bundle files are not present in the skill directory, so navigation breaks at the first hop. | 2 / 3 |
Total | 9 / 12 Passed |