Skip to content

.aidigestminify#35

Merged
khromov merged 15 commits into
mainfrom
aidigestminify
Aug 23, 2025
Merged

.aidigestminify#35
khromov merged 15 commits into
mainfrom
aidigestminify

Conversation

@khromov

@khromov khromov commented Aug 22, 2025

Copy link
Copy Markdown
Owner

No description provided.

@khromov khromov requested a review from Copilot August 23, 2025 12:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 the minify feature in v1.5.0, allowing files to be included with placeholder content instead of full content through .aidigestminify patterns. The feature includes CLI support, library API options, and watch mode compatibility.

  • Adds .aidigestminify file support for including files with placeholder descriptions
  • Implements --minify-file CLI option and library callback customization
  • Refactors codebase to separate concerns with new types.ts and digest.ts files

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/utils.ts Removes Ignore type import and adds new utility functions moved from index.ts
src/types.ts New file defining TypeScript types for IgnoreInstance, MinifyFileDescriptionCallback, and ProcessedFile
src/index.ts Major refactor moving core functionality to digest.ts, keeping only CLI and public API
src/index.test.ts Adds comprehensive tests for minify functionality and updates existing tests for code style
src/digest.ts New file containing core processing logic moved from index.ts with minify support
package.json Version bump to 1.5.0 and adds ESLint/Husky development dependencies
eslint.config.js New ESLint configuration with TypeScript support
README.md Extensive documentation updates for minify functionality and library usage
CLAUDE.md Updates development guidance with minify feature details
CHANGELOG.md Documents v1.5.0 changes
.husky/pre-commit Adds pre-commit hook for lint-staged
.github/workflows/test.yml Adds linting step to CI workflow
.aidigestignore Excludes scripts directory from codebase dumps

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

Comment thread src/digest.ts Outdated
Comment thread src/digest.ts
Comment on lines +881 to +882
minifyFile: string = ".aidigestminify",
): Promise<void> {

Copilot AI Aug 23, 2025

Copy link

Choose a reason for hiding this comment

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

The default parameter should be defined in the function signature's options parameter with other defaults, not as a separate parameter with a default value. This creates inconsistency with other optional parameters.

Suggested change
minifyFile: string = ".aidigestminify",
): Promise<void> {
minifyFile: string,
): Promise<void> {
if (minifyFile === undefined) {
minifyFile = ".aidigestminify";
}

Copilot uses AI. Check for mistakes.
khromov and others added 4 commits August 23, 2025 14:56
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@khromov khromov merged commit 10cabf1 into main Aug 23, 2025
1 check passed
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