fix: restore missing altimate_change end markers and update test expectations

- Add back `// altimate_change end — session start telemetry` and
  `// altimate_change end — accumulate session metrics` markers
  (the removed duplicates were actually balancing real start blocks)
- Update `classifySkillTrigger` tests to expect `"unknown"` for
  unrecognized/missing trigger values (matches the behavior fix)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: anandgupta42

packages/opencode/src/session/prompt.ts

@@ -809,6 +809,7 @@ export namespace SessionPrompt {
           }
         }
         // altimate_change end — task intent classification
+        // altimate_change end — session start telemetry
       }
 
       // Ephemerally wrap queued user messages with a reminder to stay on track
@@ -958,6 +959,7 @@ export namespace SessionPrompt {
         pendingError = null
       }
       // altimate_change end — quality signal + tool chain + error fingerprints
+      // altimate_change end — accumulate session metrics
 
       // altimate_change start — detect plan file creation after tool calls
       if (agent.name === "plan" && !planHasWritten) {

packages/opencode/test/telemetry/plan-skill-telemetry.test.ts

@@ -10,9 +10,9 @@ describe("telemetry.classifySkillTrigger", () => {
     expect(Telemetry.classifySkillTrigger(undefined)).toBe("llm_selected")
   })
 
-  test("returns 'llm_selected' when extra has no trigger field", () => {
-    expect(Telemetry.classifySkillTrigger({})).toBe("llm_selected")
-    expect(Telemetry.classifySkillTrigger({ foo: "bar" })).toBe("llm_selected")
+  test("returns 'unknown' when extra has no trigger field", () => {
+    expect(Telemetry.classifySkillTrigger({})).toBe("unknown")
+    expect(Telemetry.classifySkillTrigger({ foo: "bar" })).toBe("unknown")
   })
 
   test("returns 'user_command' when extra.trigger is 'user_command'", () => {
@@ -27,9 +27,9 @@ describe("telemetry.classifySkillTrigger", () => {
     expect(Telemetry.classifySkillTrigger({ trigger: "llm_selected" })).toBe("llm_selected")
   })
 
-  test("returns 'llm_selected' for unrecognized trigger values", () => {
-    expect(Telemetry.classifySkillTrigger({ trigger: "something_else" })).toBe("llm_selected")
-    expect(Telemetry.classifySkillTrigger({ trigger: 42 })).toBe("llm_selected")
+  test("returns 'unknown' for unrecognized trigger values", () => {
+    expect(Telemetry.classifySkillTrigger({ trigger: "something_else" })).toBe("unknown")
+    expect(Telemetry.classifySkillTrigger({ trigger: 42 })).toBe("unknown")
   })
 })