fix(core): tree.json unintentionally includes telemetry metadata

[mrgrain]

Reason for this change

The ConstructInfo interface is used both for telemetry analytics (sent to AWS) and for the tree.json output. Previously, telemetry metadata was being added to ConstructInfo, which caused it to leak into tree.json - a file that should only contain basic construct information (fqn and version).

Description of changes

This PR separates the concerns by:

1. Keeping ConstructInfo clean with only fqn and version - this is the public contract for tree.json
2. Introducing a new ConstructAnalytics interface that extends ConstructInfo with optional telemetryMetadata
3. Moving the analytics-specific logic from runtime-info.ts to a new stack-metadata.ts file
4. The telemetry metadata is now only collected and used within the analytics pipeline, never touching tree.json

The feature flag ENABLE_ADDITIONAL_METADATA_COLLECTION continues to control whether telemetry metadata is included in the analytics sent to AWS.

Description of how you validated changes

• Existing unit tests updated and passing
• Added new test for metadata type filtering
• Moved jsii-dependent tests to framework-integ package where they can properly test against compiled aws-cdk-lib

Checklist

• My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

(This review is outdated)

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[mrgrain]

split-out from runtime-info.test.ts

[mrgrain]

adjusted and moved into metadata-resource.test.ts

[mrgrain]

@Mergifyio update

[mergify]

update

✅ Branch has been successfully updated

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[mergify]

Merge Queue Status

✅ The pull request has been merged at 039741f

This pull request spent 6 seconds in the queue, with no time running CI.
The checks were run in-place.

Required conditions to merge

• #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • fix(core): tree.json unintentionally includes telemetry metadata #36748
• #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • fix(core): tree.json unintentionally includes telemetry metadata #36748
• any of [🛡 GitHub branch protection]:
  • check-success = validate-pr
  • check-neutral = validate-pr
  • check-skipped = validate-pr
• any of [🛡 GitHub branch protection]:
  • check-success = build
  • check-neutral = build
  • check-skipped = build

[github-actions]

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.