Skip to content

Add agent diagnostic telemetry vocabulary - #38

Merged
ANcpLua merged 2 commits into
mainfrom
agent/agent-diagnostic-snapshots
Aug 14, 2026
Merged

Add agent diagnostic telemetry vocabulary#38
ANcpLua merged 2 commits into
mainfrom
agent/agent-diagnostic-snapshots

Conversation

@ANcpLua

@ANcpLua ANcpLua commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Register the fixed qyl.agent.diagnostic.snapshot event.
  • Add nine value-free diagnostic summary attributes and five workflow-correlation attributes.
  • Generate incubating constants, event names, registry resources, and analyzer facts from the qyl registry.
  • Keep dynamic variable/check names as data values, never telemetry keys.
  • Align the repository with .NET SDK 10.0.400, NuGet 7.9, and the 4.2.0 package line.

Stack role

Telemetry-vocabulary foundation for:

Validation

The final tree passed:

  • PR CI: SDK-400 restore/Release build, attribute hash and generated-tree checks, analyzer pipeline tests, and source-generation tests
  • merged-main CI: the same gates plus package creation and packed-artifact smoke
  • Local deterministic checks: VerifyAttributesHash, emit_attributes.py --check, emit_analyzer_registry.py --check, emit_registry_resources.py --check, and verify_deprecated_catalog.py

A local Release build attempt was canceled after it stopped producing output; it is not counted as validation.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 4125dd24-5daa-4eb6-b2f6-de27cb41b4c8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ANcpLua
ANcpLua force-pushed the agent/agent-diagnostic-snapshots branch from 83c9e61 to d0185c7 Compare August 14, 2026 17:25
@ANcpLua
ANcpLua marked this pull request as ready for review August 14, 2026 17:28
@ANcpLua
ANcpLua merged commit 45c8016 into main Aug 14, 2026
3 checks passed
@ANcpLua
ANcpLua deleted the agent/agent-diagnostic-snapshots branch August 14, 2026 17:29

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d0185c73af

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

"description": "qyl-owned vocabulary, first-class next to the pinned upstream registries (qyl architecture, loop 1). Attributes here are qyl.* names the platform emits and the collector must recognize; scope_names are the ActivitySource/Meter names qyl constructs; event_names are qyl-owned Activity or DiagnosticSource event names. Consumed by emit_analyzer_registry.py (QYL0200/QYL0201 allowlists). Entries under qyl.mcp.* are the experimental staging namespace: deletion-targeted on every upstream registry bump that lands an equivalent.",
"attributes": [
{
"key": "qyl.agent.diagnostic.extension.id",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Feed the qyl vocabulary into every registry projection

When consumers use the advertised source-generation, public-registry, or TypeSpec paths, none of these new entries are visible: RegistryLoader, emit_registry_resources.py, and emit_typespec_keys.py read only resolved-registry.json, which contains no qyl.* catalog rows or the new event. Consequently, generating the qyl prefix produces no diagnostic/workflow constants, OpenResolvedRegistry() omits the vocabulary, and the qyl-api-schema projection cannot expose these correlation keys. Merge qyl-registry.json into the shared projection rather than loading it only in the attribute and analyzer emitters.

AGENTS.md reference: AGENTS.md:L3-L5

Useful? React with 👍 / 👎.

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.

1 participant