Skip to content

Update scanWorkspaceDocs any[] type to vscode.Uri[]#33

Merged
AlexJSully merged 2 commits intomainfrom
alexjsully-260214-any-to-vscodeuri
Feb 15, 2026
Merged

Update scanWorkspaceDocs any[] type to vscode.Uri[]#33
AlexJSully merged 2 commits intomainfrom
alexjsully-260214-any-to-vscodeuri

Conversation

@AlexJSully
Copy link
Owner

No description provided.

Copy link
Contributor

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

This PR updates type annotations in the workspace scanner to improve type safety by replacing any[] types with vscode.Uri[]. It also includes dependency updates for development packages and minor documentation clarifications.

Changes:

  • Replaced any[] return type with vscode.Uri[] in scanWorkspaceDocs function
  • Removed as any type assertions and replaced with explicit vscode.Uri types in filter callbacks
  • Updated regex patterns for path splitting (stylistic change, functionally equivalent)
  • Updated development dependencies to latest patch/minor versions
  • Simplified documentation to remove unused configuration settings

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/scanner/workspaceScanner.ts Changed return type from any[] to vscode.Uri[], removed type assertions, updated regex patterns
package.json Updated devDependencies to latest versions (@types/node, @typescript-eslint/*, esbuild)
package-lock.json Lockfile updates corresponding to package.json changes
docs/project/design-doc.md Removed unused settings from example config, simplified scanner description

@AlexJSully AlexJSully merged commit fc14d3d into main Feb 15, 2026
2 checks passed
@AlexJSully AlexJSully deleted the alexjsully-260214-any-to-vscodeuri branch February 15, 2026 01:38
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