feat(js): allow naming generates

[marcelfolaron]

Implements #3378 custom span names for generate steps. New optional parameter called stepName can be set to define the step name displayed in traces. Will default to "generate" if stepName is not set.

Checklist (if applicable):

• PR title is following https://www.conventionalcommits.org/en/v1.0.0/
• Tested (manually, unit tested, etc.)
• Docs updated (updated docs or a docs bug required)

[marcelfolaron]

Missed to commit my change to rename the parameter name in the testapp to stepName. Should build now

[marcelfolaron]

Huh, anything I need to do here re python checks? Seems the failed check is unrelated?
@pavelgj

[pavelgj]

Huh, anything I need to do here re python checks? Seems the failed check is unrelated? @pavelgj

The data model in common is shared across runtimes (js, go and python). There are code generators that handle this stuff.

Run

pnpm build
py/bin/generate_schema_typing

(the py script may require uv)

[marcelfolaron]

pnpm build
py/bin/generate_schema_typing

I see, thank you. That worked and I committed the 2 additional file changes.

[marcelfolaron]

It's failing with license checks. When I run it locally it fails as well and running ./bin/add_license fixes it. However that adds a lot of license content to quite a few files which seems outside the scope of this PR.

[pavelgj]

It's failing with license checks. When I run it locally it fails as well and running ./bin/add_license fixes it. However that adds a lot of license content to quite a few files which seems outside the scope of this PR.

must be upstream change in the license checker... can you try adding *.toml to ignore list in

genkit/bin/check_license

Line 60 in 9d5e4ed

-ignore '**/node_modules/**/*' \

[marcelfolaron]

It's failing with license checks. When I run it locally it fails as well and running ./bin/add_license fixes it. However that adds a lot of license content to quite a few files which seems outside the scope of this PR.

must be upstream change in the license checker... can you try adding *.toml to ignore list in

genkit/bin/check_license

Line 60 in 9d5e4ed

-ignore '**/node_modules/**/*' \

That worked. Though I had to add toml and nix files

  -ignore '**/*.toml' \
  -ignore '**/*.nix' \