fix: add capability check to perf daemon handler

wangcz · claude · wangcz · commit 4890f7355883 · 2026-06-04T11:03:50.000+08:00
The perf handler was missing an isCommandSupportedOnDevice check,
so it returned empty metrics instead of UNSUPPORTED_OPERATION for
platforms where perf is not supported (e.g. harmonyos).

Co-Authored-By: Claude Opus 4.7 &lt;noreply@anthropic.com&gt;
diff --git a/src/daemon/handlers/session-observability.ts b/src/daemon/handlers/session-observability.ts
@@ -74,6 +74,9 @@ async function handlePerfCommand(params: ObservabilityParams): Promise<DaemonRes
   if (!session) {
     return errorResponse('SESSION_NOT_FOUND', 'perf requires an active session. Run open first.');
   }
+  if (!isCommandSupportedOnDevice('perf', session.device)) {
+    return errorResponse('UNSUPPORTED_OPERATION', 'perf is not supported on this device');
+  }
 
   try {
     return {

Original file line number	Diff line number	Diff line change
`@@ -74,6 +74,9 @@ async function handlePerfCommand(params: ObservabilityParams): Promise<DaemonRes`
`74`	`74`	`if (!session) {`
`75`	`75`	`return errorResponse('SESSION_NOT_FOUND', 'perf requires an active session. Run open first.');`
`76`	`76`	`}`
	`77`	`+ if (!isCommandSupportedOnDevice('perf', session.device)) {`
	`78`	`+ return errorResponse('UNSUPPORTED_OPERATION', 'perf is not supported on this device');`
	`79`	`+ }`
`77`	`80`
`78`	`81`	`try {`
`79`	`82`	`return {`