Skip to content

Feature/filter option - #20

Merged
flowernotfound merged 6 commits into
masterfrom
feature/filter-option
Aug 29, 2025
Merged

Feature/filter option#20
flowernotfound merged 6 commits into
masterfrom
feature/filter-option

Conversation

@flowernotfound

Copy link
Copy Markdown
Owner

Filter option.
Replace logger.

@flowernotfound
flowernotfound requested a review from Copilot August 28, 2025 16:25
@flowernotfound flowernotfound self-assigned this Aug 28, 2025
@flowernotfound flowernotfound added the enhancement New feature or request label Aug 28, 2025

This comment was marked as outdated.

@flowernotfound
flowernotfound requested a review from Copilot August 28, 2025 16:28

Copilot AI left a comment

Copy link
Copy Markdown

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 introduces a new --filter option for partial text matching and replaces the existing logging system with user-friendly output methods.

  • Added --filter option for partial text matching (case-insensitive) as an alternative to prefix filtering
  • Replaced internal logger methods with user-facing userError and userInfo methods for cleaner output
  • Enhanced error handling with contextual usage messages for filter-related errors

Reviewed Changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/utils/logger.ts Added userError and userInfo methods for user-facing output
src/types/environment.ts Added partial filter type to FilterConfig union
src/core/filter.ts Implemented partial match filtering and updated filter logic
src/constants/messages.ts Added partial filter messages and updated help text
src/cli/parser/types.ts Added filter_requires_value error type and filterValue field
src/cli/parser/parser.ts Implemented --filter option parsing with validation
src/cli/handlers/mainHandler.ts Updated to use new logger methods and support filter conflicts
src/cli/errors/handlers.ts Updated error handling to use userError with contextual messages
src/cli/config.ts Added filter to supported options and help text
README.md Updated documentation with filter examples
Test files Comprehensive test coverage for new functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/cli/handlers/mainHandler.ts
Comment thread src/cli/parser/parser.ts Outdated
@flowernotfound
flowernotfound merged commit 3d33052 into master Aug 29, 2025
4 checks passed
@flowernotfound
flowernotfound deleted the feature/filter-option branch August 29, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants