[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code - #7922
[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code#7922CometActions wants to merge 1 commit into
Conversation
⏱️ pre-commit per-hook timing
⏭️ 41 skipped (no matching files changed)
|
| description: "Retrieve prompt version. When project_name is supplied the lookup\ | ||
| \ is scoped to that project: a prompt belonging to a different project is\ | ||
| \ never matched, and an unknown project name returns 404. Legacy prompts with\ | ||
| \ no project are still resolved as a deprecated fallback, signalled by the\ | ||
| \ X-Opik-Deprecation response header." |
There was a problem hiding this comment.
searchPrompts can return another project’s version
The released TypeScript searchPrompts caller omits projectName from retrievePromptVersion({ name }) after workspace-wide listing, so duplicate prompt names can resolve to another project’s version and be returned under the configured project — should we pass the resolved project name or listed prompt’s project ID, or explicitly retain the documented legacy semantics?
Want Baz to fix this for you? Activate Fixer
Other fix methods
Prompt for AI Agents
Before applying, verify this suggestion against the current code. In
`sdks/code_generation/fern/openapi/openapi.yaml` around lines 7035-7039, update the
TypeScript `searchPrompts` implementation that calls `retrievePromptVersion` to honor
the documented project-scoped lookup. Resolve the configured project name before
retrieval and pass it as `projectName` (or pass the project ID from the listed prompt)
instead of calling with only `{ name }`, and add or update a regression test covering
duplicate prompt names across projects.
| description: "Retrieve prompt version. When project_name is supplied the lookup\ | ||
| \ is scoped to that project: a prompt belonging to a different project is\ | ||
| \ never matched, and an unknown project name returns 404. Legacy prompts with\ | ||
| \ no project are still resolved as a deprecated fallback, signalled by the\ | ||
| \ X-Opik-Deprecation response header." |
There was a problem hiding this comment.
Missing required AI disclosure metadata
The PR description omits the exact ## AI-WATERMARK section required by .agents/skills/documentation/SKILL.md, so this Fern/OpenAPI documentation change doesn't meet the repository PR-template requirement — should we add the completed yes/no disclosure and, when applicable, tools, models, scope, and human-verification details?
Want Baz to fix this for you? Activate Fixer You can also update your AI coding guidelines based on this comment by apply pr to [branch name]
Other fix methods
Prompt for AI Agents
Before applying, verify this suggestion against the current code. In
apps/opik-documentation/documentation/fern/openapi/opik.yaml around lines 7035-7039,
update the associated pull request description to comply with
.agents/skills/documentation/SKILL.md. Add the exact `## AI-WATERMARK` section, complete
the required yes/no disclosure, and include the tools, models, scope, and
human-verification details when applicable; do not alter the OpenAPI description solely
to address this checklist finding.
da121b5 to
7bbba47
Compare
Details
Automated daily update of OpenAPI spec and Fern code after running
./scripts/generate_openapi.shand./sdks/opik_optimizer/scripts/generate_fern_docs.py.Change checklist
Issues
N/A
Testing
Documentation