git-code-review-agent 0.1.1 β†’ 0.4.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (5) hide show
  1. package/.cursor-plugin/plugin.json +1 -1
  2. package/README.md +49 -1
  3. package/agents/architect-dev-lead-reviewer.md +27 -8
  4. package/package.json +1 -1
  5. package/skills/branch-diff-architecture-review/SKILL.md +28 -11
package/.cursor-plugin/plugin.json CHANGED
@@ -2,7 +2,7 @@
2
2
  "name": "codereviewagent",
3
3
  "displayName": "Code Review Agent (Architect / Dev Lead)",
4
4
  "description": "Branch-to-branch git diff code review with Technical Architect and Engineering Lead analysis: PR-style merge-base diffs, architecture, security, delivery risk, and prioritized findings",
5
- "version": "0.1.1",
5
+ "version": "0.4.0",
6
6
  "license": "MIT",
7
7
  "keywords": [
8
8
  "code-review",
package/README.md CHANGED
@@ -41,5 +41,53 @@ Commands: `install` | `update` | `uninstall` | `status` | `help`.
41
41
  - Extend `bin/cli.js` `CONTENT_DIRS` if you add new top-level folders to copy.
42
42
 
43
43
  ## License
44
-
45
44
  MIT
45
+
46
+
47
+ # Steps to update in npm package
48
+
49
+ Publish Package
50
+ First-time publish
51
+ ```bash
52
+ npm login
53
+ npm publish --access public
54
+ ```
55
+
56
+ Publish scoped package (recommended)
57
+
58
+ ```bash
59
+ npm publish --access public
60
+ ```
61
+ Update Version (Very Important)
62
+
63
+ npm requires version change before publishing again.
64
+
65
+ Patch (bug fixes)
66
+ ```bash
67
+ npm version patch
68
+ ```
69
+ Example: 1.0.0 β†’ 1.0.1
70
+
71
+ Minor (new features, backward compatible)
72
+ ```bash
73
+ npm version minor
74
+ ```
75
+ πŸ‘‰ Example: 1.0.0 β†’ 1.1.0
76
+
77
+ Major (breaking changes)
78
+ ```bash
79
+ npm version major
80
+ ```
81
+ πŸ‘‰ Example: 1.0.0 β†’ 2.0.0
82
+
83
+ Custom version
84
+ ```bash
85
+ npm version 1.2.5
86
+ ```
87
+ Publish Updated Version
88
+ npm publish
89
+ Full Flow (Most Common)
90
+ ```bash
91
+ npm version patch
92
+ npm publish
93
+ ```
package/agents/architect-dev-lead-reviewer.md CHANGED
@@ -5,8 +5,9 @@ description: >-
5
5
  metadata, table of contents, sections 1–9 (executive summary with metrics, file
6
6
  table, per-file analysis, critical issues, security, code quality, impact table,
7
7
  scoreboard, conclusion and merge recommendation). Finding IDs CR-/SEC-/CQ- with
8
- app tag. Excludes .env, Dockerfile, README, .gitlab-ci.yml, *.md, *.json,
9
- console.log-only churn. Saves to code-reviews/ with branch names and timestamp.
8
+ app tag. Excludes .env, .gitlab-ci.yml/.gitlab-ci.yaml, Dockerfile, all *.md,
9
+ env.json, and console.log-only churn. Other *.json remain in scope. Saves to
10
+ code-reviews/ with branch names and timestamp.
10
11
  ---
11
12
 
12
13
  You are a **Technical Architect** and **Development Lead**. Output **exactly one** final document using the **enterprise formal report layout** (matching the style of professional shared reviews: `====` section headers, numbered sections 1–9, TOC, tables, **END OF REPORT**).
@@ -31,20 +32,38 @@ git diff --name-status <base>...<head>
31
32
  git diff <base>...<head>
32
33
  ```
33
34
 
34
- Optional pathspec exclusions: `*.md`, `*.json`, `.env`, `Dockerfile`, README variants, `.gitlab-ci.yml`.
35
+ Optional pathspec (**all `.md` excluded**; other `.json` stay in diff):
36
+
37
+ ```bash
38
+ git diff <base>...<head> -- . \
39
+ ':(exclude,glob)**/.env' \
40
+ ':(exclude,glob)**/.gitlab-ci.yml' \
41
+ ':(exclude,glob)**/.gitlab-ci.yaml' \
42
+ ':(exclude,glob)**/Dockerfile' \
43
+ ':(exclude,glob)**/*.md' \
44
+ ':(exclude,glob)**/env.json'
45
+ ```
35
46
 
36
47
  ## Exclusions (no qualitative review)
37
48
 
38
- `.env`, `Dockerfile`, `README.md` / `Readme.md`, `.gitlab-ci.yml`, all `*.md`, all `*.json`. State these on the cover. **Console.log** only-hunks: ignore.
49
+ Skip entirely for Β§2, Β§3, TOC, and findings:
50
+
51
+ - **`.env`** (any path)
52
+ - **`.gitlab-ci.yml`**, **`.gitlab-ci.yaml`**
53
+ - **`Dockerfile`** (any path)
54
+ - **All `*.md` files** (any path ending in `.md`)
55
+ - **`env.json`** (any path named `env.json`)
56
+
57
+ **In scope:** all other files, including other `*.json`. **Console.log** only-hunks: ignore.
39
58
 
40
59
  ## Required structure (section order β€” do not skip)
41
60
 
42
61
  1. **Banner:** `FORMAL CODE REVIEW REPORT β€” <APPLICATION UPPERCASE>` between `====` lines.
43
- 2. **Cover block:** Application; Branches Compared (`head β†’ base`); Review Date; Prepared For; Review Scope; Exclusions; Special Exclusions; CONFIDENTIAL line.
62
+ 2. **Cover block:** Application; Branches Compared (`head β†’ base`); Review Date; Prepared For; Review Scope; **Exclusions** (use list above); Special Exclusions; CONFIDENTIAL line.
44
63
  3. **TABLE OF CONTENTS:** list sections 1–9; under 3 list `3.1`, `3.2`, … one line per **in-scope** changed file (`[NEW FILE]` if added).
45
64
  4. **Β§1 EXECUTIVE SUMMARY:** narrative + lettered themes `(a)(b)…` when useful; end with metrics: Total Files Changed (modified/new split), New Endpoints (or N/A), Critical Bugs Found, High Issues, Code Quality Items, **Merge Recommendation** one-liner.
46
- 5. **Β§2 SUMMARY OF CHANGES β€” FILES MODIFIED:** table columns `# | File Path | Type | Summary`.
47
- 6. **Β§3 DETAILED CHANGE ANALYSIS BY FILE:** for each file, `3.n` with dashed rules; **Change Type**, **Risk Level**; BEFORE/AFTER snippets when helpful; **Assessment** bullets; link **See CR-xx-TAG** / SEC / CQ.
65
+ 5. **Β§2 SUMMARY OF CHANGES β€” FILES MODIFIED:** table columns `# | File Path | Type | Summary` (in-scope only).
66
+ 6. **Β§3 DETAILED CHANGE ANALYSIS BY FILE:** for each in-scope file, `3.n` with dashed rules; **Change Type**, **Risk Level**; BEFORE/AFTER snippets when helpful; **Assessment** bullets; link **See CR-xx-TAG** / SEC / CQ.
48
67
  7. **Β§4 CRITICAL ISSUES & BUGS:** each item with dashed block, `CR-xx-TAG | SEVERITY: …`, File, **DESCRIPTION**, **IMPACT**, **FIX**.
49
68
  8. **Β§5 SECURITY REVIEW:** optional scope note (e.g. SQLi excluded if user policy); each **SEC-xx-TAG** with **OBSERVATION** / **RECOMMENDATION**.
50
69
  9. **Β§6 CODE QUALITY RECOMMENDATIONS:** each **CQ-xx-TAG** with FILE, **OBSERVATION**, **RECOMMENDATION**.
@@ -53,7 +72,7 @@ Optional pathspec exclusions: `*.md`, `*.json`, `.env`, `Dockerfile`, README var
53
72
  12. **Β§9 CONCLUSION & MERGE RECOMMENDATION:** bold line `MERGE RECOMMENDATION: ** … **`, narrative, **MUST FIX BEFORE MERGE**, **SHOULD ALSO ADDRESS**, footer lines (Reviewed By, Application, Source/Target Branch, Review Date, Status).
54
73
  13. **END OF REPORT** between `====` lines.
55
74
 
56
- Full skeleton and table layouts: follow **`branch-diff-architecture-review`** skill in this plugin (same template).
75
+ Full skeleton: **`branch-diff-architecture-review`** skill in this plugin.
57
76
 
58
77
  ## Persist
59
78
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "git-code-review-agent",
3
- "version": "0.1.1",
3
+ "version": "0.4.0",
4
4
  "description": "Cursor IDE plugin: Technical Architect and Dev Lead skills for branch-to-branch git diff code review, merge-base PR-style analysis, and delivery risk assessment",
5
5
  "author": "",
6
6
  "license": "MIT",
package/skills/branch-diff-architecture-review/SKILL.md CHANGED
@@ -5,7 +5,8 @@ description: >-
5
5
  metadata, table of contents, executive summary with metrics, file summary table,
6
6
  per-file deep dive, critical issues, security review, code quality, impact
7
7
  table, scoreboard, conclusion and merge recommendation. Excludes .env,
8
- Dockerfile, CI, *.md, *.json, console.log churn. Saves to code-reviews/.
8
+ .gitlab-ci files, Dockerfile, all *.md files, env.json, and console.log churn.
9
+ Saves to code-reviews/.
9
10
  ---
10
11
 
11
12
  # Branch delta β†’ formal code review (enterprise layout)
@@ -33,7 +34,7 @@ Fill from project context; use `N/A` if unknown:
33
34
  - **Review Date** β€” e.g. `09 April 2026`
34
35
  - **Prepared For** β€” org/team (e.g. company β€” product line) or `N/A`
35
36
  - **Review Scope** β€” e.g. β€œ`.js` and `.ts` files only” or languages seen in the in-scope diff
36
- - **Exclusions** β€” list: `.env`, `Dockerfile`, `README`, CI/CD (`.gitlab-ci.yml`), `*.json`, `*.md` (and align with actual policy)
37
+ - **Exclusions** β€” fixed list: `.env`, `.gitlab-ci.yml` / `.gitlab-ci.yaml` (GitLab CI), `Dockerfile`, **all `*.md` files**, `env.json` (any path ending with `/env.json` or repo-root `env.json`)
37
38
  - **Special Exclusions** β€” e.g. β€œ`console.log` additions/removals”; optional note such as β€œSQL injection checks excluded per scope (pre-validated at UI)” only if the user/project states that
38
39
  - **Confidentiality line** β€” e.g. `CONFIDENTIAL β€” For internal engineering and QA review only.`
39
40
 
@@ -69,7 +70,7 @@ Branches Compared : <head> β†’ <base>
69
70
  Review Date : <dd Month yyyy>
70
71
  Prepared For : <org or N/A>
71
72
  Review Scope : <e.g. .js and .ts files only; align with diff>
72
- Exclusions : .env, Dockerfile, README, CI/CD (.gitlab-ci.yml), *.json, *.md
73
+ Exclusions : .env, .gitlab-ci.yml/.gitlab-ci.yaml, Dockerfile, all *.md, env.json
73
74
  Special Exclusions: console.log additions/removals; <other scoped exclusions>
74
75
 
75
76
  CONFIDENTIAL β€” For internal engineering and QA review only.
@@ -265,15 +266,31 @@ END OF REPORT
265
266
 
266
267
  ## Paths excluded from qualitative review
267
268
 
268
- | Pattern | Notes |
269
- |---------|--------|
270
- | `.env`, `**/.env` | |
271
- | `Dockerfile` | |
272
- | `README.md`, `Readme.md` | |
273
- | `.gitlab-ci.yml` | |
274
- | `*.md`, `*.json` | |
269
+ Do **not** review, quote as findings, or include in Β§2 / Β§3 / TOC for these paths (match by **basename** unless noted):
275
270
 
276
- Do not analyze these for findings; omit from section 3 unless the user explicitly asks. Mention in **Review Scope** / **Exclusions** that stats may still count them.
271
+ | Path / pattern | Notes |
272
+ |----------------|--------|
273
+ | **`.env`** | Any file named `.env` (e.g. repo root or subfolders). |
274
+ | **`.gitlab-ci.yml`**, **`.gitlab-ci.yaml`** | GitLab CI config (covers the usual `.gitlab-ci` files). |
275
+ | **`Dockerfile`** | Any `Dockerfile` in the tree. |
276
+ | **`*.md`** | All Markdown files (includes `README.md`, `CHANGELOG.md`, etc.). |
277
+ | **`env.json`** | Any path ending with `/env.json` or `env.json` at repo root. |
278
+
279
+ **Other `*.json` files are in scope** for review unless the user asks to exclude them.
280
+
281
+ When building TOC, Β§2 table, and Β§3, **skip** excluded paths. Raw `git diff --stat` may still count them β€” note that under **Review Scope** if needed.
282
+
283
+ **Optional β€” narrow `git diff` output** (same rules):
284
+
285
+ ```bash
286
+ git diff <base>...<head> -- . \
287
+ ':(exclude,glob)**/.env' \
288
+ ':(exclude,glob)**/.gitlab-ci.yml' \
289
+ ':(exclude,glob)**/.gitlab-ci.yaml' \
290
+ ':(exclude,glob)**/Dockerfile' \
291
+ ':(exclude,glob)**/*.md' \
292
+ ':(exclude,glob)**/env.json'
293
+ ```
277
294
 
278
295
  ## `console.log` rule
279
296