Repository navigation
Update canvas-authoring external plugin to 0.1.2 - #5391
alexweininger wants to merge 1 commit into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
🔒 PR Risk Scan ResultsScanned 2 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
🚦 Submission status: 🛠️ Requires submitter fixesRisk tier: Why this tier
Automated checks
Action needed
Review
Commands
Updated for |
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
✅ External plugin PR checks passed
Per-plugin quality summary
Gate output detailscanvas-authoring - spec compliance (✅ pass)canvas-authoring - vally lint (✅ pass)canvas-authoring - install smoke test (✅ pass)canvas-authoring - version match (pass)canvas-authoring - ref/sha consistency (not_run)canvas-authoring - canvas structure (not_run) |
There was a problem hiding this comment.
🔵 Needs a closer look
Organization SAML restrictions prevented verification of the pinned upstream package and its reported telemetry changes.
1 open issue
What changed in this PR
Updates the existing canvas-authoring external plugin listing to release 0.1.2, keeping the catalog and generated marketplace entry aligned.
Changes:
- Bumps the version from
0.1.1to0.1.2and pins upstream commit1cf279cfdc7105517d24f34cab56583aed734e75. - Mirrors the version and source SHA in the generated marketplace.
| File | Description |
|---|---|
plugins/external.json |
Updates the canonical plugin version and pinned source SHA. |
.github/plugin/marketplace.json |
Synchronizes the generated marketplace entry. |
Review effort: Max
| @@ -430,7 +430,7 @@ | |||
| { | |||
| "name": "canvas-authoring", | |||
| "description": "Toolkit setup and Azure or counter starters for the host’s native `create-canvas` skill; skill-only, not a canvas extension. Includes prebuilt packages and guidance for styles, sign in, subscriptions, telemetry, and more.", | |||
aaronpowell
left a comment
There was a problem hiding this comment.
Thanks @alexweininger for disclosing the telemetry. 1) external.json still says 'skill-only, not a canvas extension', but 0.1.2 adds an extension; please update the description and regenerate marketplace.json. 2) The README says 'pseudonymous account identifier', but the value sent is the numeric GitHub user ID; please state that plainly and document the endpoint.
|
Thanks for calling out both issues. We are including canvas-authoring in the shared telemetry opt-out correction: the public Usage data section will explicitly name the numeric GitHub user ID and ingestion endpoint, and document CANVAS_TELEMETRY_DISABLED=1 rather than relying on a test-mode flag. We will also correct the catalog description to reflect the registered skill-telemetry extension and regenerate marketplace.json; installing the companion still does not itself open a canvas. We will verify that wording against the corrected package rather than retain the inaccurate skill-only/no-extension claim. A new upstream release and verified production squash pin are needed for the opt-out. This is an in-progress response, not a claim that either requested change is resolved or the package has been released. |
|
The source correction is now in draft review: coreai-microsoft/canvases-cloud-foundation#448 covers the common skill-collector opt-out and an isolated fresh-export regression; coreai-microsoft/canvases-cloud-foundation#446 fixes the public description and Usage data notice. Canvas Authoring is a skill plugin with a background extension used for its skill-usage observer; it opens no canvas and registers no tools/canvases/hooks. The disclosure names the numeric GitHub ID, actual endpoint and startup opt-out rather than implying anonymous collection or an unverified retention promise. These are source PRs, not a released replacement for the pinned plugin. Final combined qualification, required reviews and fresh production packaging are still pending. Canvas Authoring Awesome Copilot source.sha remains pending production squash merge; the catalog description and pin must be corrected against the verified release rather than presented as already fixed. |
|
Closing this as #5985 bumped the plugin to |

Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
Update the existing
canvas-authoringexternal plugin listing from0.1.1to0.1.2, pinned to production squash commit1cf279cfdc7105517d24f34cab56583aed734e75from microsoft/azure-dev-tools#37.Only the version and source SHA change in
plugins/external.jsonand the regenerated.github/plugin/marketplace.json; listing copy and unrelated entries are unchanged. The published package adds invocation-count telemetry through a canvas-less session extension.Type of Contribution
Additional Notes
Validation:
npm run plugin:generate-marketplace— passed; only the targeted marketplace entry changed.npm run plugin:validate— passed; all 106 local plugins and the external catalog are valid.npm run build— passed; generated output is up to date, with no additional changes.bash eng/fix-line-endings.sh— passed; no additional changes.git diff --check— passed.npm startwas not run separately (npm run buildis its configured implementation). End-to-end plugin installation was not run locally; PR external-plugin quality gates provide the install smoke test andvally lintchecks. This is an existing catalog update, not a new resource submission.Gateway caveat: live collection is not claimed. The updated telemetry gateway deployment remains blocked at preflight on the undocumented Azure table schema key
isTroubleshootingAllowed, pending service-owner clarification. This PR does not deploy Azure resources or change release refs.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.