Skip to content

Conversation

Copy link

Copilot AI commented Nov 26, 2025

Replace // DEPRECATED - use original comments with proper JSDoc @deprecated tags so IDEs and documentation tools surface the deprecation warning. The annotations clarify that the import path is deprecated, not the exported classes themselves.

Changes

  • src/builders/index.node.ts - FromNodePackageJson re-export
  • src/factories/index.common.ts - LicenseFactory, PackageUrlFactory re-exports
  • src/factories/index.node.ts - FromNodePackageJson re-export
  • src/types/index.ts - NodePackageJson types re-export
  • src/utils/index.node.ts - NpmjsUtility, LicenseUtility re-exports

Example

/**
 * @deprecated This re-export location is deprecated.
 * Import from `Contrib.FromNodePackageJson.Builders` instead.
 * The exported classes themselves are NOT deprecated - only this import path.
 */
export * as FromNodePackageJson from '../contrib/fromNodePackageJson.node/builders'

Includes merged changes from PR #1343 (contrib namespace refactor).

Original prompt

see those re-exports marked with DEPRECATED - use original.
suggest codeannotation to make it clear that not the original exported class is deprecated, but the use that is exported there - since the code was moved.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits November 26, 2025 13:07
Replace simple `// DEPRECATED - use original` comments with proper JSDoc
@deprecated tags. The annotations clearly indicate that:
1. The re-export location is deprecated (not the class/type itself)
2. Users should import from the new Contrib namespace instead
3. The exported classes/types themselves are NOT deprecated

This makes deprecation visible in IDEs and documentation tools.

Co-authored-by: jkowalleck <2765863+jkowalleck@users.noreply.github.com>
Copilot AI changed the title [WIP] Add code annotation for deprecated re-exports Add JSDoc @deprecated annotations for contrib re-exports Nov 26, 2025
Copilot AI requested a review from jkowalleck November 26, 2025 13:16
Copilot finished work on behalf of jkowalleck November 26, 2025 13:16
@jkowalleck jkowalleck changed the base branch from main to feat/prep-contrib November 26, 2025 13:21
@jkowalleck jkowalleck closed this Nov 27, 2025
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.

2 participants