Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JS] missing uuid import on 0.9 RC #1184

Open
cabljac opened this issue Nov 5, 2024 · 1 comment
Open

[JS] missing uuid import on 0.9 RC #1184

cabljac opened this issue Nov 5, 2024 · 1 comment
Assignees
Labels
bug Something isn't working js

Comments

@cabljac
Copy link
Contributor

cabljac commented Nov 5, 2024

Describe the bug

Running a simple flow in Next.js (in a test) it complained about missing uuid.

I'm not 100% this is a genkit issue, but my flow was just this:

import { z } from "genkit";
import { ai } from "../genkit";

export const mainFlow = ai.defineFlow(
  {
    name: "mainFlow",
    inputSchema: z.string(),
    outputSchema: z.string(),
  },
  async (input) => input
);

and my test:

import { describe, it, expect } from "vitest";
import { mainFlow } from "./mainFlow";

describe("mainFlow", () => {
  it("should run", async () => {
    const result = await mainFlow("hello");
    expect(result).toBe("hello");
  });
});

Expected behavior
Not to need to install uuid as a dependency, probably

Screenshots
Screenshot 2024-11-05 at 20 17 41

Runtime (please complete the following information):

  • OS: [MacOS]

** Node version
❯ node --version
v20.11.1

Additional context
This is on the 0.9 RC

@cabljac
Copy link
Contributor Author

cabljac commented Nov 12, 2024

We found we needed to add uuid during testing in this update as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working js
Projects
Status: No status
Development

No branches or pull requests

2 participants