chore: strip implementation comments and remove an orphaned API report - #59
Open
eordano wants to merge 1 commit into
Open
chore: strip implementation comments and remove an orphaned API report#59eordano wants to merge 1 commit into
eordano wants to merge 1 commit into
Conversation
Token-reduction pass, independent of the toolchain modernization. - Remove implementation comments from 41 source and test files, and from pnpm-workspace.yaml and the two workflows. Public JSDoc on exported API is kept, since it reaches consumers through the emitted .d.ts, as are functional directives (eslint-disable and friends). Every file here is a comment-only change: re-running the strip over each file's previous revision reproduces the new contents byte-for-byte, so no behaviour is affected. - Delete libs/urn-resolver/etc/urn-resolver.api.md, an API Extractor report with no generator left in the repo. Nothing produces or checks it, so it was silently drifting stale. Stripping the workflow comments removes the rationale for pinning actions by commit SHA, and stripping pnpm-workspace.yaml removes the explanation of why allowBuilds and publicHoistPattern list what they do. Both were verified to be comment-only with no behavioural effect, but that reasoning is now recorded nowhere. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011Kio4QZqSB6LcjcjQbxWZy
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Token-reduction pass
Stripping the workflow comments removes the rationale for pinning actions by commit SHA, and stripping pnpm-workspace.yaml removes the explanation of why allowBuilds and publicHoistPattern list what they do. Both were verified to be comment-only with no behavioural effect, but that reasoning is now recorded nowhere.