Repository navigation
chore: remove the @path editor and the agent and tool keys nothing reads - #129
Merged
Merged
Conversation
No producer has a `@path` decorator: wfpy's `path` is an argument of `@viewer`. The property panel's `@path` editor never rendered, and the annotation wizard's template wrote `@path(paths=[...])`, which makes the Python source fail to import. Both go; a `path` annotation a project defines itself falls through to the generic editor like any custom annotation. Also removed, each with no reader anywhere in the platform or the products: `wf:toolSpec` on tool nodes (the external-actor flag it set alongside stays), `cal:agentStateful` and `WorkflowDiagramMetadata.AGENT_STATEFUL`, the model storage's never-called `clearAgentContextMetadata`, and the property panel's `createTabs` and `inlineMarkdown`. First step of moving wfpy's own features out of the platform: what has no reader is deleted before the rest moves to wfpy-ide. Claude-Session: https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv
- The agent-context section appended to a tab only the removed `createTabs` built; it now appends to the panel content directly, which is where it always landed. - The tab stylesheet rules (and their theme overrides) styled nothing once `createTabs` was gone; removed, with their rows in the selector baseline. - `renderInlineMarkdownSafe` had one caller, the removed `inlineMarkdown`. - The agent-context entry types still declared `stateful`, now unread. - The changelog gave the wrong reason for the `@path` template: wfpy does set tool search paths, with `@config(path=[...])`; the template wrote a decorator that does not exist for it. - The agent reserved-key list is indented as the expression it now is. Claude-Session: https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
First step of moving wfpy's own features out of the platform and into wfpy-ide: what has no reader at all is deleted before the rest moves.
What goes
@pathannotation editor (property panel) and its wizard template. No producer has a@pathdecorator: wfpy sets a workflow's tool search paths with@config(path=[...])(wfpy/core.py:1524). The editor never rendered. The wizard's "external tool search paths" template wrote@path(paths=[...])into the source, which the Python source then cannot import. Apathannotation a project defines itself falls through to the generic key/value editor like any custom annotation.wf:toolSpecon tool nodes. Nothing reads it; the external-actor flag set alongside it stays.cal:agentStatefulandWorkflowDiagramMetadata.AGENT_STATEFUL: written by the agent-context overlay, read by nothing.clearAgentContextMetadatain the model storage: never called.createTabsandinlineMarkdownin the property panel: never called; with them the tab stylesheet rules andrenderInlineMarkdownSafe, which only they used.AGENT_STATEFULleaves the shared API, so the next release's minor bump (already due for #127) covers it.No product reads any of these (checked wfpy-ide, streamblocks-ide, sv-ide, vhdl-ide, ndf-ide, chisel-ide and mlir-viewer).
Left for later:
scriptInterpreterCommandsis also unread, but two products set it and #124 moves the type that declares it, so it is deprecated after #124 lands rather than here.Checks
scripts/typecheck.sh: 5/5 packages;scripts/check-neutrality.sh: 5/5 gates.variant-node-create-wizard.test.ts, which fails on untouched main too and passes in CI).https://claude.ai/code/session_01EToj6RJRMSSHQiqGNMU9wv