refactor: optimize tech stack — 10x smaller bundle, zero heavy deps#5
Merged
doanbactam merged 9 commits intomasterfrom Mar 27, 2026
Merged
refactor: optimize tech stack — 10x smaller bundle, zero heavy deps#5doanbactam merged 9 commits intomasterfrom
doanbactam merged 9 commits intomasterfrom
Conversation
Replace chalk/ora with ANSI escape codes, add email-domain AI pattern for better false-positive reduction. Logger reduced from 128 to 62 lines. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove listr2 task lists and prompts dependency. Use built-in readline/promises for interactive input. Sequential async flow replaces Listr wrappers. Remove unused main() exports. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove chalk, ora, listr2, prompts from dependencies - Add ESLint + Prettier with typescript-eslint config - Fix build: single bun build --target node instead of dual - Fix CI: trigger on master (not main), add PR check job - Sync VERSION to 1.0.5 - Add dist/ and *.tgz to .gitignore - Remove dead files: .noco-config.json, pnpm-lock.yaml, package-lock.json Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Delete .noco-config.json (unused), nococli-1.0.1.tgz, extra lockfiles (package-lock.json, pnpm-lock.yaml), and old per-file dist outputs. Only dist/cli.js bundle remains. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This is a test. Co-Authored-By: Claude <noreply@anthropic.com>
The generated git hook now uses Node.js (fs + RegExp) instead of bash/sed. This makes the hook work on Windows where sed is not available by default. - Hook uses #!/usr/bin/env node instead of #!/bin/bash - Patterns use JS RegExp with 'i' flag (replaces [Cc][Oo] char class hacks) - Trailing blank line cleanup via while-loop instead of sed :a ba - E2E tests updated: cross-platform GIT_SEQUENCE_EDITOR via Node.js - Unit tests: 27 pass (added regression test for commit body keywords) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
CI type-check failed: getGitUserEmail was used but not imported. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
CI format-check failed on src/cli.ts after import fix. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.
Summary
master(wasmain), added PR check job (lint/test/build)Changes
Pre-Landing Review
branches: [main]but repo usesmasterawaitonrunInstallCommand({})(fire-and-forget)Test plan
bun test— 25/25 passbun run build— 84.63 KB bundlebun run lint— 0 errors, 0 warningsnode dist/cli.js --version→1.1.0node dist/cli.js patterns→ lists 2 pattern groupsnode dist/cli.js status→ shows install status