Skip to content

Wr/spec default value @W-21157829@#317

Merged
EstebanRomero84 merged 17 commits intomainfrom
wr/specDefaultValue
Feb 4, 2026
Merged

Wr/spec default value @W-21157829@#317
EstebanRomero84 merged 17 commits intomainfrom
wr/specDefaultValue

Conversation

@WillieRuemmele
Copy link
Contributor

@WillieRuemmele WillieRuemmele commented Feb 3, 2026 •
edited
Loading

What does this PR do?

adds --no-spec to choose the default spec programmatically

What issues does this PR fix or reference?

@W-21157829@

@WillieRuemmele WillieRuemmele requested a review from a team as a code owner February 3, 2026 15:53
jshackell-sfdc
jshackell-sfdc reviewed Feb 3, 2026
View reviewed changes
jshackell-sfdc
jshackell-sfdc reviewed Feb 3, 2026
View reviewed changes
Copy link
Contributor

@jshackell-sfdc jshackell-sfdc left a comment

Choose a reason for hiding this comment

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

round 2 :).

WillieRuemmele and others added 9 commits February 3, 2026 12:38
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
76df41d 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele
docs: remove description
7fe4ff9
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
6a601b2 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
c58a4d3 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele
chore: --spec none -> --no-spec
6ee89f6
@WillieRuemmele
chore: snapshot
0b4338a
@WillieRuemmele
chore: regen yarn.lock
8e43ef7
@WillieRuemmele
chore: merge main, resolve conflicts
3a822af
@WillieRuemmele
chore: regen yarn.lock
3f12756
jshackell-sfdc
jshackell-sfdc reviewed Feb 4, 2026
View reviewed changes
Copy link
Contributor

@jshackell-sfdc jshackell-sfdc left a comment

Choose a reason for hiding this comment

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

See my edits!

WillieRuemmele and others added 4 commits February 4, 2026 08:37
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
d3e1dbc 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
adf420a 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele @jshackell-sfdc
Update messages/agent.generate.authoring-bundle.md
6e21fbc 
Co-authored-by: Juliet Shackell <63259011+jshackell-sfdc@users.noreply.github.com>
@WillieRuemmele
chore: merge main, resolve conflicts
5e5a1f9
@EstebanRomero84
Copy link
Contributor

EstebanRomero84 commented Feb 4, 2026

QA Notes

I verified following test cases for agent generate authoring-bundle:

  • --no-spec: it does not prompt for a path and uses the default boilerplate to generate the AAB ✅
  • --spec: it uses the provided yaml file to generate the AAB ✅
  • --spec with different case: it finds the correct yaml file ✅
  • --no-spec --spec: it displays an error message saying both flags can't be used together ✅
  • no flags for spec are provided: it prompt to select a yaml file ✅

@EstebanRomero84 EstebanRomero84 merged commit a47e6c6 into main Feb 4, 2026
28 of 29 checks passed
@EstebanRomero84 EstebanRomero84 deleted the wr/specDefaultValue branch February 4, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jshackell-sfdc jshackell-sfdc jshackell-sfdc left review comments

@EstebanRomero84 EstebanRomero84 EstebanRomero84 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@WillieRuemmele @EstebanRomero84 @jshackell-sfdc