Devesh/sk 2832 claude setup v2 - #338

Open

Devesh-Skyflow wants to merge 56 commits into

mainfrom

devesh/SK-2832-claude-setup-v2

Commits on May 29, 2026

SK-2832 add Claude Code setup (CLAUDE.md, commands, workflows)

Devesh-Skyflow
and
claude
committed
9d8775b
SK-2832 remove release-v1.yml — not part of Claude setup
Devesh-Skyflow
committed
08df1ab
SK-2832 split samples context into samples/CLAUDE.md

Devesh-Skyflow
and
claude
committed
1819b05
SK-2832 add basic samples reference to root CLAUDE.md
Devesh-Skyflow
committed
c8b52b0
SK-2832 make commit guidelines directive for Claude
Devesh-Skyflow
committed
50fe8bf
SK-2832 replace redundant commit rules with /commit reference
Devesh-Skyflow
committed
c3ec30f
fix: example
Devesh-Skyflow
committed
9de9e92
SK-2832 rename /test to /quality, add 100% coverage requirement
Devesh-Skyflow
committed
a9647ce
SK-2832 reference /quality in code-review final verdict
Devesh-Skyflow
committed
1b54cee
SK-2832 add /quality check step to /commit before committing
Devesh-Skyflow
committed
d3dc74b
SK-2832 enforce /commit rule in CLAUDE.md, add /quality to review skill
Devesh-Skyflow
committed
390b513
SK-2832 require /quality before code-review, not just after
Devesh-Skyflow
committed
2f1eac1
SK-2832 add descriptive header to checkstyle-on-edit.py
Devesh-Skyflow
committed
80a5ebf
SK-2832 remove paths from review skill, delegate all rules to code-review.md
Devesh-Skyflow
committed
fedad62
SK-2832 restore paths to review skill for proactive review triggers
Devesh-Skyflow
committed
cadbca9
Merge remote-tracking branch 'origin/main' into devesh/SK-2832-claude-setup-v2
Devesh-Skyflow
committed
eceba8f
SK-2832 add devesh to cspell wordlist
Devesh-Skyflow
committed
967c3c9
SK-2832 suppress cspell on branch name example, revert devesh from wordlist
Devesh-Skyflow
committed
ed287b1

Commits on Jun 1, 2026

SK-2832 update the claude setup and remove duplicate things
skyflow-bharti
committed
e437f44
SK-2832 Rename the commit md file to git-commit
skyflow-bharti
committed
d571bef
SK-2832 Removed the duplicate section from commands
skyflow-bharti
committed
5ad91d6
SK-2832 Removed the duplicate section from skill
skyflow-bharti
committed
10b523e
SK-2832 Removed the duplicate claude setup file from samples
skyflow-bharti
committed
72e0853
SK-2832 Removed the unused code patterns
skyflow-bharti
committed
b8fb38b

Commits on Jun 2, 2026

SK-2832: Updated pr review workflow
aadarsh-st
committed
976079b
SK-2832: Fix claude issue
aadarsh-st
committed
10e1c09
SK-2832: fix
aadarsh-st
committed
7953daa
SK-2832: check api key valid or not
aadarsh-st
committed
ec3e412
SK-2832: Debug
aadarsh-st
committed
93f8bd5
SK-2832: Updated the token count
aadarsh-st
committed
e409f0a
SK-2832: Updated condition for code-review
aadarsh-st
committed
376bdef

Commits on Jun 5, 2026

SK-2832: scope Claude PR review to changed lines, consolidate output, inline comments

Devesh-Skyflow
and
claude
committed
e0736e5
SK-2832: fail Claude PR review on API errors, drop redundant security job

Devesh-Skyflow
and
claude
committed
c1bf5fc
SK-2832: drop serviceaccount severity tag from code-review

Devesh-Skyflow
and
claude
committed
fba0b3e

Commits on Jun 8, 2026

SK-2832: fix severity taxonomy in PR review (Quality is advisory, not blocking)

Devesh-Skyflow
and
claude
committed
97f1ea4
SK-2832: split review taxonomy into one Severity scale + separate Category

Devesh-Skyflow
and
claude
committed
9238c56
SK-2832: tidy AI review output (rename, crisp summary, no preamble, wider cols)

Devesh-Skyflow
and
claude
committed
87ef55f
SK-2832: fix AI review markdown rendering (flat blocks, robust preamble strip)

Devesh-Skyflow
and
claude
committed
09c4cce
SK-2832: make inline-findings block leak-proof (HTML sentinel, no backtick fence)

Devesh-Skyflow
and
claude
committed
1975a35
SK-2832: Updated cache write
aadarsh-st
committed
1b07efb
Merge branch 'devesh/SK-2832-claude-setup-v2' of https://github.com/skyflowapi/skyflow-java into devesh/SK-2832-claude-setup-v2
aadarsh-st
committed
7ea857d
SK-2832: incremental + debounced Claude PR review

Devesh-Skyflow
and
claude
committed
667c4e0
SK-2832: remove redundancy in AI review output

Devesh-Skyflow
and
claude
committed
24338f8
SK-2832: add ReviewProbe to test PR review (intentional bugs)

Devesh-Skyflow
and
claude
committed
8bbdc6c
SK-2832: add dummy comment block to ReviewProbe (incremental silence test)

Devesh-Skyflow
and
claude
committed
25ecaf2
SK-2832: dedupe the verdict line in AI review output

Devesh-Skyflow
and
claude
committed
d944382
SK-2832: exclude generated/ from PR review changed-lines metric

Devesh-Skyflow
and
claude
committed
c80e811
SK-2832: stream Claude review output live to the Actions log

Devesh-Skyflow
and
claude
committed
205011d
SK-2832: swap probe bugs to test incremental new-issue detection

Devesh-Skyflow
and
claude
committed
49ea994
SK-2832: add intentional advisory smells to probe (magic number, commented-out code)

Devesh-Skyflow
and
claude
committed
1cf756f
SK-2832 Segregate code-review PR/CI output spec into .claude/includes/code-review-ci.md

Devesh-Skyflow
and
claude
committed
fb8901d
SK-2832: swap probe to a fresh set of bugs and smells

Devesh-Skyflow
and
claude
committed
c612922
SK-2832 CI review: drop verdict for count summary, add incremental re-review note

Devesh-Skyflow
and
claude
committed
ee83966

Commits on Jun 9, 2026

SK-2832 TEMP: trigger PR review on setup-branch base for probe testing

Devesh-Skyflow
and
claude
committed
24f7a83

Commits on Jun 11, 2026