Skip to content

Add tests for unique non-exported helpers and update symbols#124

Merged
Looted merged 5 commits intomasterfrom
develop
Mar 28, 2026
Merged

Add tests for unique non-exported helpers and update symbols#124
Looted merged 5 commits intomasterfrom
develop

Conversation

@Looted
Copy link
Copy Markdown
Owner

@Looted Looted commented Mar 28, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 28, 2026 19:28
@Looted Looted merged commit 5744d73 into master Mar 28, 2026
8 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds regression coverage for the TypeScript symbol-coordinate matcher’s fallback paths (unique internal helpers and class methods) and rolls the CLI patch release metadata forward after consuming the related changesets. Also refreshes generated symbol coordinate timestamps in the repo’s symbols manifest.

Changes:

  • Add Bun tests asserting coordinates are generated for a unique non-exported top-level function and for unique (including private) class methods.
  • Bump kibi-cli to 0.4.1 and add the corresponding CHANGELOG.md entry (with the applied changesets removed).
  • Refresh generated coordinatesGeneratedAt fields in documentation/symbols.yaml.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/cli/tests/extractors/symbols-ts.test.ts Adds coverage for matcher fallback resolution to unique internal functions and class methods.
packages/cli/package.json Bumps CLI package version to 0.4.1.
packages/cli/CHANGELOG.md Adds 0.4.1 release notes for the patch release.
documentation/symbols.yaml Updates generated symbol coordinate timestamps after a refresh.
.changeset/kibi-cli-resolve-kb-pl-path.md Removed as part of applying/versioning changesets.
.changeset/add-fallback-kibi-cli.md Removed as part of applying/versioning changesets.

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