-
Notifications
You must be signed in to change notification settings - Fork 153
Add 1DS telemetry for power-pages with default-on opt-out #127
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
Closed
Closed
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
5febefe
docs: add design spec for 1DS telemetry infra
fa0e782
docs: add 1DS telemetry implementation plan
e46d074
docs(telemetry): drop 1DS SDK; adopt detached-dispatcher hooks
b1e761d
docs(plan): revise 1DS telemetry plan for dispatcher-based design
942d262
feat(telemetry): scaffold shared/telemetry directory
96cfdad
feat(telemetry): add per-process session id helper
8676d3b
feat(telemetry): add consent read/write with schema-version gating
38487c6
feat(telemetry): add pre→post correlation via OS temp file
2d9b00c
feat(telemetry): add no-op scrubber placeholder
26d6131
feat(telemetry): add strict-allowlist event builders
a6506a0
feat(telemetry): add standalone emit-dispatcher child CLI
77b1d3b
feat(telemetry): harden emit-dispatcher fail-closed guarantees
2a99ddb
feat(telemetry): add emit-spawn helper for detached dispatcher
78a3094
feat(telemetry): harden emit-spawn env scope + use execPath
bbdab5c
feat(telemetry): add withTelemetry wrapper using fireAndForget
b46a0ed
feat(telemetry): add check-consent CLI
bbc050b
feat(telemetry): add record-consent CLI
d696c95
feat(telemetry): add sync-to-plugin script
1ed23e8
docs(telemetry): add consent reference doc
ddc4052
feat(power-pages): sync shared telemetry library into plugin
54e619a
feat(power-pages): add PreToolUse:Skill telemetry hook
4da89a1
feat(power-pages): emit skill_completed from existing PostToolUse hook
249cd9e
feat(power-pages): register PreToolUse:Skill telemetry hook
d1fdcbd
feat(power-pages): add Phase-1 telemetry-consent check to create-site
675fccd
feat(power-pages): add Phase-1 telemetry-consent check to remaining t…
7dd1b5c
feat(power-pages): add telemetry-runner helper for script instrumenta…
6967545
feat(power-pages): instrument check-activation-status with withTelemetry
600e5c0
feat(power-pages): instrument verify-dataverse-access with withTelemetry
60758b4
feat(power-pages): instrument render-audit-report with withTelemetry
8b1242e
feat(power-pages): instrument clear-site-cache with withTelemetry
3131652
feat(power-pages): instrument skill validators with runInstrumented
67f7410
docs(power-pages): document telemetry conventions
5dcd8b6
docs: document shared telemetry convention at repo level
16a39ed
docs(telemetry): add shared library README
6dec0fb
docs: link telemetry notice from root README
66eb7bf
feat(telemetry): append events to local JSONL when iKey is placeholder
347852a
docs(telemetry): add slash-command telemetry design spec
9903bc3
docs(telemetry): note semantic gap in skill_completed on programmatic…
682b6f8
docs(telemetry): add implementation plan for slash-command telemetry
02ecf67
feat(telemetry): add prompt-detector for slash-command skill detection
e9329c1
feat(telemetry): add emit-from-prompt orchestrator for slash-command …
6b3dddb
test(telemetry): assert prompt-detector and emit-from-prompt are synced
04ee885
test(power-pages): add integration test for user-prompt telemetry hook
a8ca02f
feat(power-pages): add UserPromptSubmit telemetry hook for slash comm…
3352d58
feat(power-pages): register UserPromptSubmit telemetry hook
8b3d93b
fix(telemetry): forward POWER_PLATFORM_SKILLS_{CONFIG_DIR,FAKE_HTTPS}…
f0f4168
chore(power-pages): sync slash-command telemetry helpers from shared
0f21d40
fix(telemetry): reshape 1DS envelope to remove rejected baseType/base…
bde5915
fix(telemetry): restore envelope.name=PagesPowerPlatformExtEvent for …
818bbac
fix(telemetry): match 1ds-core-js SDK camelCase+stringified eventInfo…
64e1e3b
fix(telemetry): set envelope.name to VscodeEvent to match tenant anno…
4ddfe79
chore(telemetry): address review findings — hook parity + spec freshness
c59716e
feat(telemetry): default-on with opt-out, drop consent prompt
6e73d3f
docs(telemetry): add plugin-owner adoption guide for 1DS telemetry
acc1746
chore(telemetry): provision real 1DS iKey + INT collector URL
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This repo-level README says telemetry is collected with "explicit user consent", but the telemetry posture described elsewhere in this PR is default-on with opt-out (no first-run prompt). Please adjust this wording to match the implemented behavior (e.g., "enabled by default with documented opt-out").