Skip to content

Conversation

WillieRuemmele
Copy link
Contributor

@WillieRuemmele WillieRuemmele commented Sep 16, 2025

DF DEMO BRANCH

main<- wr/nga <- nga

What does this PR do?

adds two commands:
agent publish authoring-bundle
agent validate authoring-bundle

What issues does this PR fix or reference?

@W-19294085@
@W-19294687@

Testing Setup

will require some new local files in the project
image
$SF_MOCK_DIR set to agent library test/mocks

@WillieRuemmele WillieRuemmele requested a review from a team as a code owner September 16, 2025 16:56

# flags.api-name.summary

Path to the Agent Authoring Bundle to publish
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Path to the Agent Authoring Bundle to publish
API name of the Agent Authoring Bundle to publish

};
} catch (error) {
// Handle validation errors
const err = error instanceof Error ? error : new Error(String(error));
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI: SfError.wrap() does this for you.

Suggested change
const err = error instanceof Error ? error : new Error(String(error));
const err = SfError.wrap(error);

};
} catch (error) {
// Handle validation errors
const err = error instanceof Error ? error : new Error(String(error));
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const err = error instanceof Error ? error : new Error(String(error));
const err = SfError.wrap(error);

import { execCmd } from '@salesforce/cli-plugins-testkit';
import type { AgentPublishAuthoringBundleResult } from '../../src/commands/agent/publish/authoring-bundle.js';

describe.skip('agent publish authoring-bundle NUTs', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove .skip now?

import { execCmd } from '@salesforce/cli-plugins-testkit';
import type { AgentValidateAuthoringBundleResult } from '../../src/commands/agent/validate/authoring-bundle.js';

describe.skip('agent validate authoring-bundle NUTs', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove .skip now?

* All rights reserved.
* Licensed under the BSD 3-Clause license.
* For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Not blocking, but this isn't the license header to use anymore.

EstebanRomero84 and others added 30 commits October 6, 2025 15:38
…File

W-19829287: if api-name flag is not used, prompt user to select an .agent file from the project
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.

5 participants