This repository was archived by the owner on Oct 1, 2026. It is now read-only.
Repository navigation
Format README code blocks for ruff 0.16 - #64
Closed
DidierRLopes wants to merge 1 commit into
Closed
DidierRLopes wants to merge 1 commit into
DidierRLopes wants to merge 1 commit into
Conversation
ruff 0.16 (installed unpinned in CI) now formats Python code blocks inside Markdown files. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KocxNUyBQT8K8qTduX7Gyu
andrewkenreich
pushed a commit
that referenced
this pull request
Aug 18, 2026
> **Stacked PR:** based on #64 (repo hygiene needed for green CI). Merge that first — GitHub retargets this PR automatically and the diff here contains only the feature. Part of the cross-repo **"save this as a skill"** feature: users can ask the copilot to save the current conversation's workflow as a reusable skill in their Workspace skill library. ## What this PR does - Adds `"save_skill"` to the closed `FunctionCallSSEData.function` Literal in `openbb_ai/models.py`, allowing agents to emit a `copilotFunctionCall` SSE that asks the Workspace client to save a skill. This follows the exact precedent of `get_skill_content` (#52). - Bumps the package version to **2.2.0**. ## Protocol flow enabled by this change 1. The agent (openbb-ada) yields `FunctionCallSSE(function="save_skill", input_arguments={"name", "instructions"})` and closes the stream. 2. The Workspace frontend generates the skill from the conversation, saves it, and re-POSTs `/query` with an `LlmClientFunctionCallResultMessage` carrying a `ClientCommandResult` (`{"skill": {"slug", "description"}}` on success). 3. The agent confirms the save to the user. ##⚠️ Version note PR #61 (durable conversation summaries) also bumps to 2.2.0. Whichever of the two merges second must bump to 2.3.0 before release. ## Companion PRs — review/test together This PR only extends the protocol; the behavior lives in the companion PRs and needs all three to test end-to-end: - openbb-ada: OpenBB-finance/openbb-ada#643 - terminalpro: OpenBB-finance/terminalpro#2106 **Release ordering:** merge + publish this package to PyPI first (ada's `^2.1.0` caret accepts 2.2.0, but ada CI/Docker installs from PyPI, so the ada PR's new Literal value fails validation until 2.2.0 is published) → then openbb-ada → terminalpro can go before or alongside ada. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01KocxNUyBQT8K8qTduX7Gyu --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Repo hygiene, no behavior change. CI installs ruff unpinned, and ruff 0.16 started formatting Python code blocks inside Markdown files — since the lint workflow only runs on PRs, main drifted and every new PR inherits the failure. This formats the README code blocks so lint is green again.
Split out of #63, which is stacked on this branch — merge this first.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KocxNUyBQT8K8qTduX7Gyu