Skip to content

Conversation

@pavelgj
Copy link
Collaborator

@pavelgj pavelgj commented Nov 19, 2025

test case in question

  it('should remain dynamic after registration', () => {
    const dynamic = tool({ name: 'dynamic', description: 'test' });
    assert.strictEqual(isDynamicTool(dynamic), true);

    registry.registerAction('tool', dynamic);

    assert.strictEqual(isDynamicTool(dynamic), true);
  });

and

  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);
  });

Checklist (if applicable):

test case in question

```ts
  it('should remain dynamic after registration', () => {
    const dynamic = tool({ name: 'dynamic', description: 'test' });
    assert.strictEqual(isDynamicTool(dynamic), true);

    registry.registerAction('tool', dynamic);

    assert.strictEqual(isDynamicTool(dynamic), true);
  });
  ```
@pavelgj pavelgj changed the title fix(js/ai): fixed dynamic tools becoming non-dynamic once registered fix(js/ai): fixed dynamic tools/resources becoming non-dynamic once registered Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants