-
Notifications
You must be signed in to change notification settings - Fork 0
DF Branch [DO NOT MERGE] #69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
WillieRuemmele
wants to merge
74
commits into
main
Choose a base branch
from
df
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
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
Wr/rename afscript to agent @W-19803943@
…io Code extensions
…updating build messages
…ild scripts, and adjust VSCode settings
…or improved theme support
…ency, introduce sample tracer data in json format
…agent selection and error handling
…ion management and communication with webview
…, enhancing user interaction and notifications
…g state reset of agent preview
…ng user experience and interaction
… session activity, improving user context awareness
…w, enhancing debugging capabilities and user feedback
…agent selection and user interaction
…iner, streamlining the component structure
…l consistency and user experience
…remove max-width from agent message content for improved layout consistency
…ed session handling and UI state management
… AgentCombinedView, update navigation group assignments in package.json for improved command organization
… for improved theming and adjust padding for better layout
…der text in ChatInput and PlaceholderContent for better user guidance
…enhance UI consistency and theming
…oss AgentPreview and AgentTracer components to enhance UI consistency and improve user experience
…view and AgentTracer for enhanced visual consistency
…g custom variables for improved consistency across AgentPreview, AgentTracer, and shared components
…ability and visual consistency
…der properties, and responsive design elements for improved layout and visual consistency
…e VSCode theme tokens for color and adjust padding for improved visual consistency
…adding, and border-radius for improved visual consistency
… visual consistency
…n AgentTracer component for improved visual consistency
…sistency and theming
…tCode and outputCode to tracer
…y esbuild configuration to disable bundling and enable sourcemaps, and clean up package.json by removing unused dependencies
…o target ES2022, and adjust esbuild configuration to exclude 'vscode' as an external dependency
…clude sample tracer data for Agent Tracer
Ml/design update
… error icons in UI
… color in StepAction CSS
Action Error for Tracer Mockup
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.
NOTE:
original PR #65 was merged, and reverted #66
this PR reopens the original changes, which have all been approved and merged once-before
DO NOT MERGE
What does this PR do?

adds an AFDX: validate afscript context option to .afscript files
What issues does this PR fix or reference?
@W-19293903@
Functionality Before
no NGA functionality
Functionality After
able to quickly validate afScript via server-side compile
Testing Setup Notes
get the built vsix from the clicking on the green checkmark by the commit hash, select build-and-test, click on summary in the upper left, scroll to the bottom for the vsix, installing this, and setting SF_MOCK_DIR should work as well
using maybe-mock in the development extension host session is overly complicated. I found the best approach was to change vscode-agents/node_modules/@salesforce/agents/lib/agent.js Line 323 to be
and an example failure response,
{
"status": "failure",
"errors": [{
"errorType": "ISSUE",
"description": "couldn't compile this",
"colStart": 1,
"colEnd": 2,
"lineStart": 1,
"lineEnd": 1
},
{
"errorType": "ISSUE123",
"description": "testing error 2",
"colStart": 12,
"colEnd": 22,
"lineStart": 12,
"lineEnd": 12
}],
Note: there's an error property already in the mock, you just need to place those correctly and update the status for a failure response