Skip to content

patch(agentfabric): Update agentfabric relevant files#52

Draft
Drake-Ash wants to merge 2 commits into
salesforce:mainfrom
Drake-Ash:udpate-agentfabric-relevant-files
Draft

patch(agentfabric): Update agentfabric relevant files#52
Drake-Ash wants to merge 2 commits into
salesforce:mainfrom
Drake-Ash:udpate-agentfabric-relevant-files

Conversation

@Drake-Ash
Copy link
Copy Markdown

What

Why

How

Test Plan

  • Existing tests pass (pnpm test)
  • New/updated tests cover the change
  • Linting and type checks pass (pnpm lint && pnpm typecheck)

Checklist

  • My code follows the project's coding style
  • I have reviewed my own diff
  • I have added/updated documentation as needed
  • This change does not introduce new warnings

sf-ashwin added 2 commits June 5, 2026 20:12
Sync the agentfabric dialect and the @agentscript/language APIs it
depends on from the internal module-agentscript repo, so the build and
test suite pass in op-agentscript.

- Add agentfabric compiler and graph-extraction modules, lint rules, and
  tests carried over from module-agentscript.
- Extend @agentscript/language with the graph field markers the dialect
  needs: transitionContainer, predicateField, outputNameField, and
  displayLabelField (core/types.ts FieldMetadata flags + field-builder
  methods). Export isAstNodeLike and the Range type.
- Update getValueCompletions to the AST-aware signature and add
  walkParentsToSchemaContext plus CompletionCandidate.insertText; thread
  cycle detection through scope.ts walkers; update the lsp caller.
- Mark agentscript-dialect baseSubagentFields.label as displayLabelField
  so orchestrator nodes (cloned from SubagentBlock) surface their label
  in the extracted graph.
@salesforce-cla
Copy link
Copy Markdown

salesforce-cla Bot commented Jun 5, 2026

Thanks for the contribution! It looks like @sf-ashwin is an internal user so signing the CLA is not required. However, we need to confirm this.

@Drake-Ash Drake-Ash marked this pull request as draft June 5, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants