@onlooker-community/ecosystem 0.43.7 → 0.44.0

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "ecosystem",
-  "version": "0.43.7",
+  "version": "0.44.0",
   "description": "Observability substrate for Claude Code. Provides the shared $ONLOOKER_DIR storage root (default $HOME/.onlooker), canonical schema-validated event emission, session and tool tracking hooks, and prompt rules. Required by all other Onlooker plugins.",
   "author": {
     "name": "Onlooker Community",

package/.release-please-manifest.json

@@ -1,5 +1,5 @@
 {
-  ".": "0.43.7",
+  ".": "0.44.0",
   "plugins/archivist": "0.3.2",
   "plugins/tribunal": "1.2.8",
   "plugins/echo": "0.3.2",
@@ -9,7 +9,7 @@
   "plugins/scribe": "0.4.3",
   "plugins/counsel": "0.5.2",
   "plugins/warden": "0.3.2",
-  "plugins/librarian": "0.13.3",
+  "plugins/librarian": "0.14.0",
   "plugins/curator": "0.2.4",
   "plugins/historian": "0.3.4",
   "plugins/assayer": "1.1.4",

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.44.0](https://github.com/onlooker-community/ecosystem/compare/ecosystem-v0.43.7...ecosystem-v0.44.0) (2026-08-23)
+
+
+### Features
+
+* **librarian:** let lessons status close the judge walk :abacus: ([#205](https://github.com/onlooker-community/ecosystem/issues/205)) ([8da1cd7](https://github.com/onlooker-community/ecosystem/commit/8da1cd72a7e564aae44750efaa1e59f1ecd59641))
+
 ## [0.43.7](https://github.com/onlooker-community/ecosystem/compare/ecosystem-v0.43.6...ecosystem-v0.43.7) (2026-08-23)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlooker-community/ecosystem",
-  "version": "0.43.7",
+  "version": "0.44.0",
   "description": "Agents, skills, hooks, commands, rules, and MCP configurations that power [Onlooker](https://onlooker.dev)",
   "author": {
     "name": "Onlooker Community",

package/plugins/librarian/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "librarian",
-  "version": "0.13.3",
+  "version": "0.14.0",
   "description": "Consolidation layer between archivist's per-session artifacts and the user's durable typed memory store. Detects which session decisions, dead-ends, and open questions deserve to live across sessions, classifies them into the user/feedback/project/reference types, and queues them as proposals for explicit confirmation. Auto-promotion is opt-in. Builds on the Onlooker ecosystem plugin.",
   "author": {
     "name": "Onlooker Community",

package/plugins/librarian/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.14.0](https://github.com/onlooker-community/ecosystem/compare/librarian-v0.13.3...librarian-v0.14.0) (2026-08-23)
+
+
+### Features
+
+* **librarian:** let lessons status close the judge walk :abacus: ([#205](https://github.com/onlooker-community/ecosystem/issues/205)) ([8da1cd7](https://github.com/onlooker-community/ecosystem/commit/8da1cd72a7e564aae44750efaa1e59f1ecd59641))
+
 ## [0.13.3](https://github.com/onlooker-community/ecosystem/compare/librarian-v0.13.2...librarian-v0.13.3) (2026-08-22)
 
 

package/plugins/librarian/scripts/lib/librarian-cli.sh

@@ -641,13 +641,44 @@ librarian_cli_lessons_defer() {
 	printf 'Deferred %s; it stays in the queue.\n' "$lesson_id"
 }
 
+# One authoritative line for the whole lesson queue.
+#
+# Reports a count per status rather than pending alone, because the judge walk
+# closes on this call. It used to close on counts the model had tallied itself
+# across the loop, with nothing to check them against — every other walk in the
+# skill ends on a CLI call, and this one had none to end on (ecosystem-sv3).
+#
+# `awaiting promotion` is the walk's fourth bucket: a lesson whose verdict is
+# recorded but whose pool entry or declined row has not landed. promote()
+# stamps promoted_at LAST, after the terminal record, so a terminal status with
+# no stamp is exactly that state — and it must not be folded into `approved`,
+# which would claim a pool entry that does not exist.
+#
+# The walk's remaining bucket, could-not-judge, is deliberately absent: such a
+# candidate stays `confirmed`, which is indistinguishable on disk from one that
+# was never judged this run. That count is run-scoped and stays the model's.
 librarian_cli_lessons_status() {
 	local cwd="${1:-}"
-	local key pending
+	local key
 	key=$(_librarian_cli_project_key "$cwd")
 	[[ -z "$key" ]] && { printf 'No project key resolvable from this directory.\n'; return 1; }
-	pending=$(librarian_lesson_list_pending "$key")
-	printf 'lessons pending: %s\n' "$(printf '%s' "$pending" | jq 'length')"
+
+	local s counts=()
+	for s in pending confirmed approved rejected; do
+		counts+=("$(librarian_lesson_list_by_status "$key" "$s" | jq 'length')")
+	done
+
+	# A terminal verdict whose promote() never stamped. Counted from the same
+	# per-status reads rather than a second directory walk.
+	local awaiting=0 n
+	for s in approved rejected; do
+		n=$(librarian_lesson_list_by_status "$key" "$s" \
+			| jq '[.[] | select(has("promoted_at") | not)] | length')
+		awaiting=$(( awaiting + n ))
+	done
+
+	printf 'lessons pending: %s, confirmed: %s, approved: %s, rejected: %s, awaiting promotion: %s\n' \
+		"${counts[0]}" "${counts[1]}" "${counts[2]}" "${counts[3]}" "$awaiting"
 }
 
 librarian_cli_lessons() {

package/plugins/librarian/skills/librarian/SKILL.md

@@ -141,10 +141,21 @@ For `lessons judge`, run the jury over confirmed candidates:
    those ids and list them at the end so the user knows to re-run. A broken judge
    must never become a rejection — the artifact's watermark has already moved,
    so a false rejection buries a good lesson permanently.
-5. Finish by reporting counts: approved, rejected, could-not-judge, and
-   **judged but not promoted** (verdict recorded, pool write still pending —
-   list these ids with `librarian_cli lessons promote <id>` as the next step).
-   Don't fold this bucket into "approved": a lesson here has no pool entry yet.
+5. **Finish on `librarian_cli lessons status`, not on your own tally.** It
+   prints one line — pending, confirmed, approved, rejected, and **awaiting
+   promotion** — read from disk, so it is the same kind of authoritative close
+   the review walk ends on. Report those numbers as it gives them; a long batch
+   is exactly where a running count kept in your head drifts.
+
+   Two things the call cannot tell you, which you must still report yourself:
+
+   - **could-not-judge** — those candidates stay `confirmed`, which on disk is
+     indistinguishable from a candidate that was never judged this run. Keep
+     the ids from step 4 and list them.
+   - **which** lessons are awaiting promotion. The call gives the count; you
+     have the ids. List them with `librarian_cli lessons promote <id>` as the
+     next step, and don't fold them into "approved" — a lesson here has no pool
+     entry yet. If your ids and the count disagree, trust the count and say so.
 
 `scope_accuracy` is the criterion that matters most on a `version_independent`
 candidate. The schema guarantees such a lesson **carries** a justification; this

package/test/bats/librarian-lesson-review.bats

@@ -387,6 +387,29 @@ _cli_setup() {
 	source "${PLUGIN_ROOT}/scripts/lib/librarian-cli.sh"
 }
 
+# ── lessons status: a count per status (ecosystem-sv3) ───────────────────────
+#
+# The judge walk used to close by reporting counts the model tracked itself
+# across the loop, with nothing to check them against — every other walk in the
+# skill ends on an authoritative CLI call. `lessons status` reported only a
+# pending count, so there was nothing to ground the summary on.
+
+# Force a proposal to a terminal status on disk. Reaching `approved` through
+# the real path means empaneling a jury, which is what the judge walk exists to
+# avoid doing twice; the status field is what the counter reads.
+_set_status() {
+	local id="$1" status="$2" promoted="${3:-}"
+	local f
+	f="$(librarian_lessons_dir "$PROJECT_KEY")/proposals/${id}.json"
+	local tmp="${f}.tmp"
+	if [[ -n "$promoted" ]]; then
+		jq --arg s "$status" --arg p "$promoted" '.status = $s | .promoted_at = $p' "$f" >"$tmp"
+	else
+		jq --arg s "$status" '.status = $s | del(.promoted_at)' "$f" >"$tmp"
+	fi
+	mv -f "$tmp" "$f"
+}
+
 @test "lessons status reports zero on an empty queue" {
 	_cli_setup
 	run librarian_cli lessons status "$PROJECT_REPO"
@@ -394,6 +417,72 @@ _cli_setup() {
 	[[ "$output" == *"0"* ]] || return 1
 }
 
+@test "lessons status reports a count for every status, not just pending" {
+	_cli_setup
+	local a b c d
+	a=$(_seed_pending)
+	b=$(_seed_pending)
+	c=$(_seed_pending)
+	d=$(_seed_pending)
+	librarian_lesson_confirm "$PROJECT_KEY" "$b" "org"
+	_set_status "$c" "approved" "2026-08-01T00:00:00Z"
+	_set_status "$d" "rejected" "2026-08-01T00:00:00Z"
+
+	run librarian_cli lessons status "$PROJECT_REPO"
+	[ "$status" -eq 0 ] || return 1
+	[[ "$output" == *"pending: 1"* ]] || return 1
+	[[ "$output" == *"confirmed: 1"* ]] || return 1
+	[[ "$output" == *"approved: 1"* ]] || return 1
+	[[ "$output" == *"rejected: 1"* ]]
+}
+
+@test "lessons status counts a judged-but-unpromoted lesson separately" {
+	# The judge walk's fourth bucket. promote() stamps promoted_at LAST, after
+	# the terminal record lands, so a terminal status with no stamp is exactly
+	# the state the walk is told not to fold into "approved" — the lesson has
+	# no pool entry yet.
+	_cli_setup
+	local a b
+	a=$(_seed_pending)
+	b=$(_seed_pending)
+	_set_status "$a" "approved" "2026-08-01T00:00:00Z"
+	_set_status "$b" "approved"
+
+	run librarian_cli lessons status "$PROJECT_REPO"
+	[ "$status" -eq 0 ] || return 1
+	[[ "$output" == *"approved: 2"* ]] || return 1
+	[[ "$output" == *"awaiting promotion: 1"* ]]
+}
+
+@test "lessons status reports awaiting promotion for a rejected lesson too" {
+	# A rejected verdict also lands its terminal record (the declined row)
+	# before the stamp, so the same recovery state exists on that side.
+	_cli_setup
+	local a b
+	a=$(_seed_pending)
+	b=$(_seed_pending)
+	_set_status "$a" "rejected"
+	# A promoted sibling, so the count cannot come out right by counting every
+	# rejected lesson — without it this passes whether or not the stamp is read.
+	_set_status "$b" "rejected" "2026-08-01T00:00:00Z"
+
+	run librarian_cli lessons status "$PROJECT_REPO"
+	[ "$status" -eq 0 ] || return 1
+	[[ "$output" == *"rejected: 2"* ]] || return 1
+	[[ "$output" == *"awaiting promotion: 1"* ]]
+}
+
+@test "lessons status reports zeros across every status on an empty queue" {
+	_cli_setup
+	run librarian_cli lessons status "$PROJECT_REPO"
+	[ "$status" -eq 0 ] || return 1
+	[[ "$output" == *"pending: 0"* ]] || return 1
+	[[ "$output" == *"confirmed: 0"* ]] || return 1
+	[[ "$output" == *"approved: 0"* ]] || return 1
+	[[ "$output" == *"rejected: 0"* ]] || return 1
+	[[ "$output" == *"awaiting promotion: 0"* ]]
+}
+
 @test "lessons list shows a pending lesson" {
 	_cli_setup
 	id=$(_seed_pending)