From c58041f2c352bde67f7ab545917c8c0759d95346 Mon Sep 17 00:00:00 2001
From: Brennan Benson <79079362+brennanb2025@users.noreply.github.com>
Date: Tue, 28 Jul 2026 15:53:05 -0700
Subject: [PATCH] test(skills): pin that a routine outdated copy raises no
 attention marker

---
 .../src/lib/skill-freshness-display-status.test.ts        | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/renderer/src/lib/skill-freshness-display-status.test.ts b/src/renderer/src/lib/skill-freshness-display-status.test.ts
index b619e59e854..f8bdf54d1b1 100644
--- a/src/renderer/src/lib/skill-freshness-display-status.test.ts
+++ b/src/renderer/src/lib/skill-freshness-display-status.test.ts
@@ -167,6 +167,14 @@ describe('getSkillFreshnessDisplayStatus', () => {
 })
 
 describe('hasSkillCopyNeedingAttention', () => {
+  it('raises no attention marker over a routine outdated copy the update converges', () => {
+    // Why: an out-of-date convergent copy is ordinary work for the update command.
+    // Amber on the review affordance there would cry wolf on every routine update.
+    expect(
+      hasSkillCopyNeedingAttention(inventory([placement('outdated')], [SKILL_NAME]), SKILL_NAME)
+    ).toBe(false)
+  })
+
   it('ignores a traversal bound but keeps a real scan fault', () => {
     expect(
       hasSkillCopyNeedingAttention(