same for resources

Author: pavelgj

js/ai/src/resource.ts

@@ -138,6 +138,7 @@ export function defineResource(
 ): ResourceAction {
   const action = dynamicResource(opts, fn);
   action.matches = createMatcher(opts.uri, opts.template);
+  action.__action.metadata.dynamic = false;
   registry.registerAction('resource', action);
   return action;
 }
@@ -193,7 +194,7 @@ export async function findMatchingResource(
 
 /** Checks whether provided object is a dynamic resource. */
 export function isDynamicResourceAction(t: unknown): t is ResourceAction {
-  return isAction(t) && !t.__registry;
+  return isAction(t) && t.__action?.metadata?.dynamic === true;
 }
 
 /**

js/ai/tests/resource/resource_test.ts

@@ -23,6 +23,7 @@ import {
   dynamicResource,
   findMatchingResource,
   isDynamicResourceAction,
+  resource,
 } from '../../src/resource.js';
 import { defineEchoModel } from '../helpers.js';
 
@@ -58,7 +59,7 @@ describe('resource', () => {
         uri: 'foo://bar',
       },
       type: 'resource',
-      dynamic: true,
+      dynamic: false,
     });
 
     assert.strictEqual(testResource.matches({ uri: 'foo://bar' }), true);
@@ -229,7 +230,7 @@ describe('resource', () => {
         uri: undefined,
       },
       type: 'resource',
-      dynamic: true,
+      dynamic: false,
     });
 
     const gotBaz = await findMatchingResource(registry, resList, {
@@ -286,4 +287,15 @@ describe('isDynamicResourceAction', () => {
       true
     );
   });
+
+  it('should remain dynamic after registration', () => {
+    const dynamic = resource({ uri: 'bar://baz' }, () => ({
+      content: [{ text: `bar` }],
+    }));
+    assert.strictEqual(isDynamicResourceAction(dynamic), true);
+
+    registry.registerAction('resource', dynamic);
+
+    assert.strictEqual(isDynamicResourceAction(dynamic), true);
+  });
 });