Greptile Summary

This PR narrows the log level for deleteSession permission failures during dreaming narrative cleanup from warn to debug, preventing log flooding during cron-driven dreaming cycles that lack operator.admin scope. The implementation is small and targeted: a regex check on the error message routes missing-scope errors to an optional debug method, with the existing warn path preserved for all other failures. Tests cover both the new path and the existing warn path.

Confidence Score: 5/5

Safe to merge — all remaining findings are P2 or lower, no correctness or reliability concerns.

The change is narrow and correct: regex matches the specific error format, optional chaining on debug is intentional for callers without a debug method, and the test covers both branches. No logic, data integrity, or security concerns.

No files require special attention.

_{Reviews (1): Last reviewed commit: "fix(memory-core): suppress WARN for narr..." | Re-trigger Greptile}

Related work from PRtags group ultimate-chow-4ei0

Title: Open PR duplicate: [Bug]: memory-core: narrative session cleanup fails with "missing scope: operator.admin"

* This PR

Closing this as duplicate or superseded after Codex automated review.

PR #68312 is a duplicate of the open maintainer-tracked PR #68020. Both target the same memory-core dreaming cleanup warning: suppress expected missing scope: operator.admin cleanup noise while preserving warnings for real cleanup failures. Current main still has the pre-change warn path, so this is not implemented-on-main; the remaining review should stay on #68020.

Best possible solution:

Close #68312 as a duplicate and keep the remaining product/code review on #68020, the open MEMBER-authored, maintainer-labeled canonical PR for this memory-core cleanup-warning behavior.

What I checked:

• Canonical related PR: Provided GitHub context shows fix(memory-core): silence expected operator.admin scope miss in dreaming cleanup #68020 is open, authored by a MEMBER, labeled maintainer, and describes the same root cause and fix: subagent.deleteSession requires operator.admin, cron/background context lacks that scope, and only the expected missing-scope cleanup warning should be suppressed.
• Duplicate grouping: The PRtags comment groups fix(memory-core): downgrade narrative cleanup WARN to debug for missing-scope errors #68312 with fix(memory-core): silence expected operator.admin scope miss in dreaming cleanup #68020, fix(memory-core): downgrade cleanup warning to debug when missing operator.admin scope #68087, fix: reduce log level for narrative session cleanup scope failures #68130, and fix(memory-core): suppress expected dreaming cleanup scope warnings #68681 under the same duplicate title: memory-core narrative session cleanup fails with missing scope: operator.admin.
• Target PR scope matches canonical work: Provided PR files show fix(memory-core): downgrade narrative cleanup WARN to debug for missing-scope errors #68312 only changes dreaming-narrative.ts and dreaming-narrative.test.ts to route /missing scopes?:/i cleanup errors to optional logger.debug instead of logger.warn, matching fix(memory-core): silence expected operator.admin scope miss in dreaming cleanup #68020's stated behavior. (f01d580fcd0d)
• Current main still has pre-change cleanup warning: The local Logger type has info, warn, and error only, and the finally block catches deleteSession cleanup errors and always logs memory-core: narrative session cleanup failed... at warn level. (extensions/memory-core/src/dreaming-narrative.ts:55, d54d2d6b9b8a)
• Scope contract remains real: Current main still requires operator.admin for sessions.delete, and the gateway plugin fallback test asserts fallback session deletion rejects with missing scope: operator.admin rather than minting admin scope. (src/gateway/method-scopes.ts:159, d54d2d6b9b8a)

So I’m closing this here and keeping the remaining discussion on the canonical linked item.

Codex Review notes: model gpt-5.5, reasoning high; reviewed against d54d2d6b9b8a.