@onlooker-community/ecosystem 0.43.6 → 0.44.0

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "ecosystem",
-  "version": "0.43.6",
+  "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,15 +1,15 @@
 {
-  ".": "0.43.6",
+  ".": "0.44.0",
   "plugins/archivist": "0.3.2",
   "plugins/tribunal": "1.2.8",
   "plugins/echo": "0.3.2",
-  "plugins/cartographer": "0.6.1",
+  "plugins/cartographer": "0.6.2",
   "plugins/governor": "0.3.3",
   "plugins/compass": "0.4.2",
   "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,19 @@
 # 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)
+
+
+### Bug Fixes
+
+* **cartographer:** drop a typeless finding instead of hashing it as "unknown" :broom: ([#203](https://github.com/onlooker-community/ecosystem/issues/203)) ([a949d5d](https://github.com/onlooker-community/ecosystem/commit/a949d5d43f3ae3515ac6278b35d1be5cc468cb0e))
+
 ## [0.43.6](https://github.com/onlooker-community/ecosystem/compare/ecosystem-v0.43.5...ecosystem-v0.43.6) (2026-08-22)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlooker-community/ecosystem",
-  "version": "0.43.6",
+  "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/cartographer/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "cartographer",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Proactive periodic auditor of the persistent instruction layer (CLAUDE.md, AGENTS.md, .claude/rules/). Discovers all instruction files in the repo, extracts semantic maps, and surfaces contradictions, shadowing, gaps, and drift before they cause expensive agent misbehavior. Builds on the Onlooker ecosystem plugin.",
   "author": {
     "name": "Onlooker Community",

package/plugins/cartographer/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to the Cartographer plugin are documented here.
 
+## [0.6.2](https://github.com/onlooker-community/ecosystem/compare/cartographer-v0.6.1...cartographer-v0.6.2) (2026-08-23)
+
+
+### Bug Fixes
+
+* **cartographer:** drop a typeless finding instead of hashing it as "unknown" :broom: ([#203](https://github.com/onlooker-community/ecosystem/issues/203)) ([a949d5d](https://github.com/onlooker-community/ecosystem/commit/a949d5d43f3ae3515ac6278b35d1be5cc468cb0e))
+
 ## [0.6.1](https://github.com/onlooker-community/ecosystem/compare/cartographer-v0.6.0...cartographer-v0.6.1) (2026-08-22)
 
 

package/plugins/cartographer/scripts/run-audit.sh

@@ -247,7 +247,29 @@ run_synthesize() {
 	while IFS= read -r finding; do
 		[[ -z "$finding" ]] && continue
 		local ftype ffile_a fexcerpt_a ffile_b fexcerpt_b
-		ftype=$(printf '%s' "$finding" | jq -r '.type // "unknown"')
+		# `// ""` collapses absent, null and empty into one check, the same way
+		# cartographer_issue_found_payload does — jq's `//` treats "" as
+		# present, so an empty type would otherwise slip past.
+		ftype=$(printf '%s' "$finding" | jq -r '.type // ""')
+
+		# A finding with no type is malformed: every analyzer writes a literal
+		# type, so this means the phase that produced it has a bug. Drop it here
+		# rather than hashing it.
+		#
+		# This used to default to "unknown". The emit path already refused to
+		# announce such a finding (ecosystem-ci0), but the orchestrator stored
+		# it anyway under a hash derived from a type it does not have, and then
+		# touched its dedup sentinel. That sentinel made the silence permanent:
+		# every later audit saw "known", took the refresh path, and emitted
+		# nothing — so the finding sat on disk forever, counted in
+		# new_finding_count, and could never be announced. Dropping it keeps the
+		# stored corpus honest and leaves the hash of every well-formed finding
+		# untouched, so no existing dedup sentinel is orphaned (ecosystem-0ty).
+		if [[ -z "$ftype" ]]; then
+			log "synthesize: dropped a finding that carries no type — the analyzer that produced it has a bug"
+			continue
+		fi
+
 		ffile_a=$(printf '%s' "$finding" | jq -r '.file_a // ""')
 		fexcerpt_a=$(printf '%s' "$finding" | jq -r '.excerpt_a // ""')
 		ffile_b=$(printf '%s' "$finding" | jq -r '.file_b // ""')

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/cartographer-run-audit.bats

@@ -268,10 +268,12 @@ _run_audit() {
 	[ "$output" = "session_start_first_run" ]
 }
 
-# A typeless finding must not vanish. The builder refuses it (ecosystem-ci0),
-# but refusing is only half the fix: if the orchestrator discards that refusal
-# the event is still silently absent, one layer further up. The audit log is
-# where an operator would go looking for it.
+# A typeless finding must not vanish. Since ecosystem-0ty the synthesize phase
+# drops it before it is hashed, so this line now comes from there rather than
+# from the emit-phase builder refusal that ecosystem-ci0 added. Both write
+# "carries no type" on purpose: what matters to an operator is that the audit
+# log says a finding was discarded and why, not which layer discarded it. The
+# builder's own refusal is still covered directly in cartographer-events.bats.
 @test "a finding with no type is reported in the audit log" {
 	cat > "${STUB_BIN}/claude" <<'STUB'
 #!/usr/bin/env bash
@@ -286,6 +288,50 @@ STUB
 	grep -q 'carries no type' "$AUDIT_LOG"
 }
 
+# Refusing to announce a typeless finding was only half of ecosystem-ci0. The
+# orchestrator still stored it under a hash computed from a type it does not
+# have, then touched its dedup sentinel — so on every later audit the sentinel
+# said "known", the refresh path ran, and no bus event ever fired again. The
+# finding sat on disk forever, counted in new_finding_count, and was
+# permanently unannounceable. These three assert it is dropped instead
+# (ecosystem-0ty).
+
+_typeless_stub() {
+	cat > "${STUB_BIN}/claude" <<'STUB'
+#!/usr/bin/env bash
+cat >/dev/null
+echo call >> "$CLAUDE_CALL_LOG"
+printf '[{"severity":"warning","file_a":"CLAUDE.md","excerpt_a":"Always prefer tabs.","description":"typeless"}]'
+STUB
+	chmod +x "${STUB_BIN}/claude"
+}
+
+@test "a finding with no type is never written to findings/" {
+	_typeless_stub
+	run _run_audit
+	[ "$status" -eq 0 ] || return 1
+	[ "$(find "${CARTOGRAPHER_DIR}/findings" -name '*.json' 2>/dev/null | wc -l | tr -d ' ')" = "0" ]
+}
+
+@test "a finding with no type leaves no dedup sentinel" {
+	# The sentinel is what made this permanent: with one on disk, a corrected
+	# finding that later hashes the same way would be seen as already known and
+	# silently refreshed instead of announced.
+	_typeless_stub
+	run _run_audit
+	[ "$status" -eq 0 ] || return 1
+	[ "$(find "${CARTOGRAPHER_DIR}/dedup" -type f 2>/dev/null | wc -l | tr -d ' ')" = "0" ]
+}
+
+@test "a finding with no type is not counted as a new finding" {
+	# audit.complete used to report a finding that produced no issue.found,
+	# so a consumer reading only the bus saw counts that its own event stream
+	# could not account for.
+	_typeless_stub
+	_run_audit
+	_last_audit_complete | jq -e '.payload.new_finding_count == 0' >/dev/null
+}
+
 
 # ── Resolution reaching the bus (ecosystem-w2i) ──────────────────────────────
 #

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)