Skip to content

OpenConceptLab/ocl_online#611 | Call canonical $invoke instead of deprecated /invoke/ - #127

Merged
snyaggarwal merged 1 commit into
mainfrom
ocl_online#611
Nov 4, 2026
Merged

snyaggarwal merged 1 commit into
mainfrom
ocl_online#611

Conversation

@snyaggarwal

Copy link
Copy Markdown
Contributor

Part of OpenConceptLab/ocl_online#611.

getPromptTemplateInvokeURL now builds ${resolvedURI}$invoke/ (default and versioned forms). The version-from-URL parser also ignores $invoke.

Depends on the $invoke endpoint (ocl-ai-assistant#180) being deployed, and should land after or with the ocl-analytics-api classifier PR. Not run locally.

🤖 Generated with Claude Code

…ration instead of deprecated /invoke/

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@snyaggarwal
snyaggarwal marked this pull request as ready for review November 4, 2026 07:47

@snyaggarwal snyaggarwal left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed together with OpenConceptLab/ocl-analytics-api#88 and OpenConceptLab/ocl-ai-assistant#241. The client builds the canonical /$invoke/ route for both default and versioned template URIs, and version extraction correctly excludes both operation segments. ESLint passes for the changed file. No blocking findings; recommend merge after the endpoint is available and after, or with, ocl-analytics-api#88.

@snyaggarwal
snyaggarwal merged commit d59499c into main Nov 4, 2026
2 checks passed
@snyaggarwal
snyaggarwal deleted the ocl_online#611 branch November 4, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant