fix: update AppOnboard skill and test files#2366
Conversation
|
@vaibbavis please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
…ig.ts and update-snapshots.js. Fixture repos (e.g. deploy-test-repo-app-service) contain real .test.js files that are part of the simulated user project — they shouldn't be discovered by the parent Jest runner.
| @@ -1,2 +1,2 @@ | |||
| --- | |||
| name: deploy | |||
There was a problem hiding this comment.
In case this hasn't been brought up before: only the top-level SKILL.md files are actually considered skills; this will only be treated as a reference file. There's no particular need to organize this like a skill.
Unlike the top-level SKILL.md files, the frontmatter will be visible to the LM when it reads this file, and it may potentially be confusing to the LM.
…ssertions and ensure no post deploy execution
|
What are the .copilot-azure/sessions folders for? I don't see the tests using the fixture files. |
There was a problem hiding this comment.
Please don't write adhoc output to the source code. I supposed that's where the session folders come from in the commit?
Description
Checklist
cd tests && npm test)npm run test:skills:integration -- <skill>)USE FOR/DO NOT USE FOR/PREFER OVERclauses: confirmed no routing regressions for competing skillsRelated Issues