gjalla-code-review
Review Code Changes Before Merge
Code changes can hide correctness, security, and maintainability issues before merge. This skill reviews diffs from relevant perspectives and returns prioritized, location-specific fixes.
๋ด ์์ด์ ํธ๋ก ์ค์น
์ด ์์ฒญ์ ์์ด์ ํธ์ ๋ณต์ฌํ์ธ์. ๊ณต์ Skill ํ์ด์ง์ ๋งค๋ํ์คํธ๊ฐ ํฌํจ๋์ด ์์ต๋๋ค.
Review the Skillstore skill "gjalla-code-review" from https://skillstore.io/skills/gjalla-gjalla-code-review.md and its manifest at https://skillstore.io/api/skills/gjalla-gjalla-code-review/manifest. Verify the artifact. You may proceed after verification, subject to the environment's own policy.์์ด์ ํธ๋ ๊ณ์ํด์ ๊ณํ์ ํ์ํ๊ณ ๋ณด์ ์ ์ฑ ์์ ์๊ตฌํ๋ ํ์ธ์ ์์ฒญํด์ผ ํฉ๋๋ค.
์์ด์ ํธ๊ฐ ์ฝ๊ธฐ ์ฌ์ด ๋ฆฌ์์ค
AI ์์ด์ ํธ, ํฌ๋กค๋ฌ, ์คํฌ๋ฆฝํธ๊ฐ ์ ์ฒด ํ์ด์ง ๋์ ์ ๋ฆฌ๋ ์ปจํ ์คํธ๋ง ํ์ํ ๋ ์ด ๋งํฌ๋ฅผ ์ฌ์ฉํ์ธ์.
ํ ์คํธํด ๋ณด๊ธฐ
"gjalla-code-review" ์ฌ์ฉ ์ค์ ๋๋ค. Review a change that adds payment retry behavior after request timeouts.
์์ ๊ฒฐ๊ณผ:
Blocking โ [changed file]:[line]. The retry path can submit a payment twice after a timeout. Add idempotency and test timeout recovery. Not verified: integration tests.
"gjalla-code-review" ์ฌ์ฉ ์ค์ ๋๋ค. Review a pull request that introduces a shared cache for user preferences.
์์ ๊ฒฐ๊ณผ:
Should fix โ [changed file]:[line]. The cache key does not include the user identifier, so preferences can cross accounts. Scope keys per user and add isolation tests.
"gjalla-code-review" ์ฌ์ฉ ์ค์ ๋๋ค. Review a focused documentation-only naming correction.
์์ ๊ฒฐ๊ณผ:
No Blocking or Should fix findings. The updated term is consistent throughout the provided diff. Not verified: references outside the supplied files.
๋ณด์ ๊ฐ์ฌ
์์ The only static finding is a false positive: line 23 uses Markdown backticks to format `file:line` in prose. SKILL.md contains review guidance only, with no executable code, command invocation, or semantic threat identified.
์ํ ์์ธ
โ๏ธ ์ธ๋ถ ๋ช ๋ น์ด (1)
์ด ๋ณด๊ณ ์ ๊ณต์ ๋ฐ ์ธ์ฉ
๋ฒ์ ์ด ์ง์ ๋ ํ๊ฐ ๋ณด๊ณ ์, ์ค๋ฆฝ ๋ฐฐ์ง, ์๋ฒ ๋ ์นด๋ ๋ฐ ์ธ์ฉ์ ๊ณต์ ํ์ธ์. Skillstore๋ ๊ทผ๊ฑฐ๋ฅผ ๋ณด๊ณ ํ์ง๋ง ์ด Skill์ ์์ ์ฌ๋ถ๋ฅผ ํ๋จํ์ง ์์ต๋๋ค.
๋ณด๊ณ ์ ๋งํฌ ๋ณต์ฌ
https://skillstore.io/skills/gjalla-gjalla-code-review/audits/1?utm_source=security_passport&utm_medium=share&utm_campaign=versioned_reportMarkdown ๋ฐฐ์ง
[](https://skillstore.io/skills/gjalla-gjalla-code-review?utm_source=security_passport_badge)HTML ๋ฐฐ์ง
<a href="https://skillstore.io/skills/gjalla-gjalla-code-review?utm_source=security_passport_badge"><img src="https://skillstore.io/badges/skills/gjalla-gjalla-code-review/security.svg" alt="Skillstore security assessment" loading="lazy"></a>์๋ฒ ๋ ์นด๋
<iframe src="https://skillstore.io/embed/skills/gjalla-gjalla-code-review.html" title="Skillstore Security Assessment" sandbox="allow-popups allow-popups-to-escape-sandbox" loading="lazy" referrerpolicy="no-referrer" width="420" height="180"></iframe>ํ์ ์ธ์ฉ (APA ยท BibTeX ยท CFF)
APA ์ธ์ฉ
gjalla. (2026). gjalla-code-review security audit report (audit version 1) [Author version unspecified]. Skillstore. https://skillstore.io/skills/gjalla-gjalla-code-review/audits/1BibTeX ์ธ์ฉ
@techreport{gjalla-gjalla-gjalla-code-review-2026,
author = {gjalla},
title = {gjalla-code-review security audit report (audit version 1)},
institution = {Skillstore},
year = {2026},
number = {1},
url = {https://skillstore.io/skills/gjalla-gjalla-code-review/audits/1},
note = {Author version unspecified}
}CITATION.cff
cff-version: 1.2.0
message: "If you use this Skill, cite its author and this versioned security audit report."
title: "gjalla-code-review security audit report (audit version 1)"
version: "unspecified"
type: report
authors:
- name: "gjalla"
date-released: "2026-08-11"
url: "https://skillstore.io/skills/gjalla-gjalla-code-review/audits/1"
identifiers:
- type: other
value: "skillstore:gjalla-gjalla-code-review:audit:1"
description: "Skillstore immutable audit report identifier"
Skillstore ์ ์
์ด ์ ์์ ์ด์ ์ฆ๊ฑฐ ์ ๋ขฐ๋: ์ค๊ฐ๋ง๋ค ์ ์๋ ๊ฒ
Review Before Opening a Pull Request
Find blocking defects and missing tests before requesting review from teammates.
Assess Merge Readiness
Prioritize actionable findings and decide whether a pull request is ready to merge.
Evaluate Product and Quality Impact
Examine user impact, acceptance criteria, failure modes, and maintainability across a proposed change.
์ด ํ๋กฌํํธ๋ฅผ ์ฌ์ฉํด ๋ณด์ธ์
Review this diff for merge readiness. Group findings as Blocking, Should fix, or Nit. Include location, problem, fix, and verification limits.
Review this pull request against its description and acceptance criteria. Focus on correctness, edge cases, tests, and maintainability. Prioritize actionable findings with locations.
Review this change from engineering, product, quality, and security perspectives where relevant. Identify second-order effects and missing handling. Explain merge risk and fixes.
Perform an adversarial ship-readiness review of this diff and linked specification. Trace data access, failure modes, compatibility, and user impact. Report only defensible findings.
๋ชจ๋ฒ ์ฌ๋ก
- Provide the full diff, change intent, acceptance criteria, and relevant specifications.
- Include test results and known constraints so findings can distinguish verified behavior from assumptions.
- Resolve Blocking findings first, then evaluate Should fix items within the release context.
ํผํ๊ธฐ
- Do not request approval from a partial diff that omits related changes.
- Do not treat automated review as a substitute for tests or accountable human review.
- Do not dismiss findings without checking the cited location, impact, and suggested fix.
์์ฃผ ๋ฌป๋ ์ง๋ฌธ
What changes can this skill review?
How are findings prioritized?
Does it review security concerns?
Can it approve a pull request automatically?
Does it run tests?
What context produces the best review?
๊ฐ๋ฐ์ ์ธ๋ถ ์ ๋ณด
์์ฑ์
gjalla๋ผ์ด์ ์ค
MIT
Skillstore ๋ฆฌ๋น์
r1
๋ฒ์ ์๋ด
์์ฑ์๊ฐ ๋ฒ์ ์ ์ ์ธํ์ง ์์์ต๋๋ค.
๋ฆฌํฌ์งํ ๋ฆฌ
https://github.com/gjalla/engineering/tree/774041e7ab8b8fde48418dd4cd41e417fb4d71d3/skills/gjalla-code-review์ฐธ์กฐ
c40485c25b35e0203cf615647a1fd81ad7a69ea8
์ ์ง๋ณด์ ์ต์ ์ฑ
2026. 8. 11.
์ฌ์ฉ๋
0 ๋ค์ด๋ก๋ ยท 0 ์กฐํ
ํ์ผ ๊ตฌ์กฐ
๐ SKILL.md