fix(server): expect derived model capabilities after provider delete

The catalog delete test asserted an exact model object from before the
catalog derived capabilities, so it only failed once both changes met on
one branch. Also declare SKIP_KEY before the ref that reads it.

Author: elkaix

apps/pythinker-web/src/components/UpdateToast.vue

@@ -12,8 +12,8 @@ const state = ref<DesktopUpdateState>();
 const busy = ref(false);
 /** Versions the user skipped. Persisted per version so the prompt does not
     return for the same build; an unnamed version is skipped for the session. */
-const skipped = ref<string[]>(readSkipped());
 const SKIP_KEY = 'pythinker.update.skipped';
+const skipped = ref<string[]>(readSkipped());
 
 let removeListener: (() => void) | undefined;
 

packages/server/test/model-catalog.e2e.test.ts

@@ -274,7 +274,15 @@ describe('model/provider catalog routes', () => {
 
     const models = await appOf(r).inject({ method: 'GET', url: '/api/v1/models' });
     expect(envelopeOf<{ items: Array<{ provider: string }> }>(models.json()).data?.items)
-      .toEqual([{ provider: 'openai', model: 'gpt4o', display_name: 'gpt-4o', max_context_size: 128000 }]);
+      .toEqual([
+        {
+          provider: 'openai',
+          model: 'gpt4o',
+          display_name: 'gpt-4o',
+          max_context_size: 128000,
+          capabilities: ['image_in', 'tool_use'],
+        },
+      ]);
   });
 
   it('maps unknown provider and model ids to catalog not-found error codes', async () => {