refactor: refocus CodeLens — archive frontend, remove garbage, keep core only#2
Merged
Merged
Conversation
…ore only What changed: - Archive frontend UI to _archive/frontend/ (components, hooks, page.tsx, layout.tsx, globals.css) - Delete debug JSON files (search1-10.json, axon_raw.json, codegraph_raw.json, emerge_raw.json, etc.) - Delete download/ folder (old release zips) - Delete 49 non-codelens skills (ASR, TTS, blog-writer, xlsx, pdf, ppt, etc.) - Archive prisma/schema.prisma to _archive/prisma/ (was template User/Post, unused) What's untouched: - Backend API routes (src/app/api/) - commandRunner.ts, normalizer.ts, all lib/ files - codelens-watch.py - skills/codelens/ (the core Python scanner) - __tests__/, examples/, mini-services/
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Wolfvin
pushed a commit
that referenced
this pull request
Jun 11, 2026
Elimination #2: Security hardening, performance optimization, and code quality - API routes: command, graph, health, node-detail, search — all hardened - constants.ts: +81 lines expanded shared constants - healthScore.ts: +67 lines improved scoring - analysisStore.ts: refactored and optimized - workspaceValidator.ts: simplified - edge_resolver.py, markdown formatter improvements - Version synced to 5.7.1
This was referenced Jun 28, 2026
Closed
Merged
Wolfvin
added a commit
that referenced
this pull request
Jul 3, 2026
Issue #170 — PR #139 needs two fixes before merge: 1. Missing @WHO/@WHAT/@PART/@entry headers on 5 new formatter files 2. Wrong command count in docs (PR #139 bumped 68→69 but added 0 commands) This commit fixes #1 by adding the standard file header to: - scripts/formatters/text.py → format_text() - scripts/formatters/vim.py → format_vim() - scripts/formatters/emacs.py → format_emacs() - scripts/formatters/junit_xml.py → format_junit_xml() - scripts/formatters/gitlab_sast.py → format_gitlab_sast() Fix #2 (wrong command count) was resolved by rebasing PR #139 on latest main (bfd7c40). The rebase took main's version of all 6 doc files (README, SKILL, SKILL-QUICK, pyproject, skill.json, graph_model.py) since PR #139's only doc changes were the wrong count bump. After rebase, sync_command_count.py --check reports clean (75 commands, matching main). Rebase conflicts resolved: - scripts/codelens.py: merged both main's 'graphml' format choice + PR #139's 5 new format choices into one add_argument() call - scripts/formatters/__init__.py: merged both main's graphml dispatch block + PR #139's 5 new format dispatch blocks into format_output() - 6 doc files: took main's version (PR #139's count bump was wrong) Verified: - 85 tests pass in test_formatters_base.py + test_formatters_phase2.py + test_command_count.py (1 deselected: test_help_lists_new_formats fails due to pre-existing -f argparse conflict on main, not a regression from this PR) - sync_command_count.py --check: clean (75 commands) - All 5 formatter modules import cleanly with new headers Findings (di luar scope, di main juga): - argparse.ArgumentError: conflicting option string: -f — main's codelens.py adds --format/-f both at subparser level (line 892) AND at global parser level (line 926). Any CLI invocation crashes. This is a pre-existing bug on main (PR #157 --diff-base introduced the global -f), NOT a regression from PR #139 or this fix. Affects test_cli.py + test_doctor.py CLI smoke tests too. Suggest separate issue to fix.
This was referenced Jul 3, 2026
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.
Refocus CodeLens as a developer tool (Python scanner + JSON generator)
Archived to _archive/frontend/
Deleted garbage
Deleted non-codelens skills (49 skills)
Archived
Untouched
518 files changed, 127,242 lines removed