fix(webapp): stop logging expected auth/restore conditions as errors

Author: d-cs

.server-changes/quiet-expected-error-logs.md

@@ -0,0 +1,13 @@
+---
+area: webapp
+type: improvement
+---
+
+Downgrade two expected, non-failure conditions from `error` to `warn` so they
+stop firing as Sentry errors:
+
+- Checkpoint restore when a RESTORE event already exists (benign idempotency
+  skip on a duplicate/retried event).
+- Token endpoint when the authorization code is invalid/expired, which is the
+  expected steady state while the CLI polls during login. Genuinely unexpected
+  failures still log at `error`.

apps/webapp/app/routes/api.v1.token.ts

@@ -37,10 +37,15 @@ export async function action({ request }: ActionFunctionArgs) {
     return json(responseJson);
   } catch (error) {
     if (error instanceof Error) {
-      logger.error("Error getting PersonalAccessToken from AuthorizationCode", {
-        url: request.url,
-        error: error.message,
-      });
+      // The CLI polls this endpoint while the user completes login, so an
+      // invalid/expired code is the expected steady state, not an error.
+      const expected = error.message === "Invalid authorization code, or code expired";
+      const fields = { url: request.url, error: error.message };
+      if (expected) {
+        logger.warn("Error getting PersonalAccessToken from AuthorizationCode", fields);
+      } else {
+        logger.error("Error getting PersonalAccessToken from AuthorizationCode", fields);
+      }
 
       return json({ error: error.message }, { status: 400 });
     }

apps/webapp/app/v3/services/restoreCheckpoint.server.ts

@@ -82,7 +82,9 @@ export class RestoreCheckpointService extends BaseService {
     });
 
     if (restoreEvent) {
-      logger.error("Restore event already exists", {
+      // Idempotency guard: the restore already happened (duplicate/retried
+      // event). This is a benign skip, not a failure.
+      logger.warn("Restore event already exists", {
         runId: checkpoint.runId,
         attemptId: checkpoint.attemptId,
         checkpointId: checkpoint.id,