📝 스킬 문서를 .claude/skills로 승격하고 university-web 리라이트 주의사항 스킬 추가 by manNomi · Pull Request #611 · solid-connection/solid-connect-web

manNomi · 2026-07-28T00:21:02Z

작업 내용

docs/skills/*.md(런북 텍스트 문서)를 .claude/skills/*/SKILL.md(YAML frontmatter 포함)로 이관해 Claude Code Skill 도구가 자동 탐지·호출할 수 있도록 승격했습니다.
apps/university-web가 별도 배포되는 Next.js Multi-Zone SSG 앱이며 현재 데스크톱 레이아웃 리라이트가 진행 중임을 명시하는 university-web-rewrite-caution 스킬을 신설했습니다. 진행 중인 worktree/브랜치 확인, 라우트 소유권 경계, AUTHENTICATION.md/COMPONENTS.md 중복 동기화, SSG 빌드 실패 설계 유지 등을 체크리스트화했습니다.
Codex와 Claude Code 양쪽에서 쓰이도록 AGENTS.md를 단일 소스로 통합했습니다. 프로젝트 개요·커밋 컨벤션·스킬 목록을 모두 AGENTS.md에 두고, CLAUDE.md는 @AGENTS.md import로 동일 내용을 가져오는 얇은 파일로 정리했습니다.
CLAUDE.md에 실수로 섞여 있던 개인 메모리 시스템 전용 [[name]] 링크 문법을 제거하고 일반 파일 경로 참조로 교체했습니다.
코드래빗 리뷰 반영: 실제 코드(packages/api-schema/scripts/sync-bruno.mjs)를 대조해 검증한 뒤 4건 모두 수정했습니다.
- univ-extends-bruno-sync: BRUNO_COLLECTION_DIR이 BRUNO_SOURCE_MODE보다 우선 적용되는 실제 동작을 반영, 존재하지 않는 develop 브랜치를 임의로 만들지 않도록 리베이스 절차 수정
- rebase-issues-405-409: 완료 검증에서 하드코딩된 커밋 개수(0 7) 기대값 제거, 조상 관계 + clean 상태 기준으로 변경
- biome-unification-ci: 검증 체크리스트에 누락된 루트 pnpm typecheck 추가
- commit-push-pr: 실행 예시 커밋 메시지에 저장소 이모지 컨벤션 반영

특이 사항

코드 동작에는 영향 없는 문서/스킬 전용 변경입니다.
.claude/skills/university-web-rewrite-caution/SKILL.md은 실제 진행 중인 리라이트 상황(브랜치/워크트리)이 시간이 지나면 바뀔 수 있으니, 병합 후 실제 상황과 계속 맞는지 주기적으로 확인이 필요합니다.

리뷰 요구사항 (선택)

.claude/skills/university-web-rewrite-caution/SKILL.md 내용이 실제 university-web 리라이트 작업 상황과 맞는지 확인 부탁드립니다.

- docs/skills/*.md를 실행 가능한 .claude/skills/*/SKILL.md 형식(frontmatter 포함)으로 이관 - apps/university-web가 별도 배포되는 Multi-Zone SSG 앱이며 현재 레이아웃 리라이트가 진행 중임을 명시하는 university-web-rewrite-caution 스킬 신설 - CLAUDE.md에 누락되어 있던 프로젝트 개요 추가 (기존엔 커밋 규칙만 존재) - AGENTS.md/llms.txt에서 새 스킬 경로와 university-web 주의사항을 참조하도록 갱신

vercel · 2026-07-28T00:21:08Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
solid-connect-university-web	Ready	Preview, Comment	Jul 28, 2026 1:26am
solid-connect-web-admin	Ready	Preview, Comment	Jul 28, 2026 1:26am
solid-connection-web	Ready	Preview, Comment	Jul 28, 2026 1:26am

coderabbitai · 2026-07-28T00:22:02Z

Walkthrough

스킬 문서 표준화
Biome 품질 검사 명령, Bruno codegen 원칙, 커밋 형식과 스킬 메타데이터를 정리했습니다.
운영 절차 문서화
리베이스 검증, Bruno 동기화, university-web 리라이트 점검 및 검증 절차를 추가했습니다.
프로젝트 참조 갱신
모노레포 구조와 커밋·스킬 규칙, university-web 관련 참조를 AGENTS.md, CLAUDE.md, llms.txt에 반영했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: wibaek, enunsnv

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	제목이 스킬 문서 승격과 university-web 주의사항 스킬 추가라는 핵심 변경을 정확히 요약합니다.
Description check	✅ Passed	관련 이슈, 작업 내용, 특이 사항, 리뷰 요구사항 섹션이 모두 있어 템플릿 요구를 대부분 충족합니다.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch docs/ai-friendliness-improvements

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4dd3f320ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-07-28T00:23:25Z

+   university 관련 worktree/브랜치가 이미 떠 있다면(예: `feat/skyscanner-layout-poc`, `fix/university-*`), 같은 화면을 건드리기 전에 해당 브랜치의 최신 diff를 확인해 중복 작업을 피한다.
+
+2. **`docs/screenshots/`에 이미 있는 POC 자료를 확인한다.**
+   - `docs/screenshots/skyscanner-layout-poc/`, `docs/screenshots/university-skyscanner-layout-poc/`에 데스크톱(1440x1000)/모바일(390x844) contact sheet가 있다.


Add the POC artifacts required by this checklist

When an agent follows this mandatory university-web checklist, neither referenced POC directory exists in the reviewed tree; a repo-wide inspection of docs/screenshots finds only gradient-before.png and gradient-after.png. The agent therefore cannot inspect the claimed visual baseline or the README referenced later in the skill, so commit those artifacts or replace these references with paths that actually exist.

AGENTS.md reference: AGENTS.md:L62-L62

Useful? React with 👍 / 👎.

- 프로젝트 개요/커밋 컨벤션/스킬 목록을 AGENTS.md 하나로 모아 Codex가 그대로 읽고 Claude Code는 @AGENTS.md import로 동일 내용을 가져오도록 변경 (중복/드리프트 제거) - CLAUDE.md에 잘못 섞여 있던 메모리 시스템 전용 [[name]] 링크 문법 제거, 일반 파일 경로 참조로 교체 - AGENTS.md Skills 표에 각 스킬의 frontmatter가 Claude Code 전용 메타데이터이며 다른 에이전트는 본문만 읽으면 된다는 안내 추가

coderabbitai

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (3)

.claude/skills/rebase-issues-405-409/SKILL.md (1)
44-53: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

1. 완료 검증의 커밋 개수를 고정하지 마세요.

origin/main이 전진하거나 브랜치에 커밋이 추가되면 정상적인 리베이스 결과도 0 8 등으로 바뀔 수 있습니다. 고정값 때문에 유효한 브랜치를 잘못 실패 처리하거나 커밋을 누락하도록 유도할 수 있으니, 조상 관계와 clean 상태를 검증하고 커밋 수는 참고값으로만 안내해 주세요.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.claude/skills/rebase-issues-405-409/SKILL.md around lines 44 - 53, Update
the “완료 검증” guidance to avoid requiring a fixed `0 7` commit count. Validate
that the working tree is clean and the rebased branch has the expected ancestor
relationship with `origin/main`; present `git rev-list --left-right --count`
only as a reference value, since the count may legitimately change.
.claude/skills/univ-extends-bruno-sync/SKILL.md (2)
48-51: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

2. BRUNO_COLLECTION_DIR 우선순위를 반영해 실패 조건을 수정하세요.

실제 packages/api-schema/scripts/sync-bruno.mjs는 BRUNO_COLLECTION_DIR이 설정되면 BRUNO_SOURCE_MODE보다 먼저 해당 로컬 경로를 사용합니다. 따라서 BRUNO_SOURCE_MODE=remote여도 유효한 BRUNO_COLLECTION_DIR이 있으면 BRUNO_REPO_URL 없이 동작할 수 있습니다.

“remote 모드에서 항상 URL이 필요하다”가 아니라, 명시적 collection directory가 없고 원격 경로를 resolve해야 할 때 URL이 필요하다고 문서화해야 합니다.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.claude/skills/univ-extends-bruno-sync/SKILL.md around lines 48 - 51, 실패 대응
섹션의 BRUNO_REPO_URL 조건을 수정해 BRUNO_COLLECTION_DIR이 유효하게 설정된 경우에는
BRUNO_SOURCE_MODE=remote여도 URL 없이 동작할 수 있음을 반영하세요. 명시적 collection directory가 없어서
원격 경로를 resolve해야 하는 경우에만 BRUNO_REPO_URL 미설정으로 즉시 실패한다고 문서화하고, 관련된
BRUNO_COLLECTION_PATH 조건은 기존 동작에 맞게 유지하세요.
15-20: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

1. develop 기준 브랜치를 실제로 최신화하는 명령을 명시하세요.

현재 절차는 git fetch --all --prune 후 로컬 develop을 바로 리베이스 기준으로 사용하지만, 기존 로컬 develop이 stale이어도 갱신하지 않습니다. develop이 없을 때 생성하는 절차도 없어 git rebase develop이 실패할 수 있습니다.

origin/develop을 사용할지, origin/main을 사용할지 명확히 정한 뒤 해당 기준을 직접 checkout/reset하거나 git rebase origin/main처럼 명시해 주세요.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.claude/skills/univ-extends-bruno-sync/SKILL.md around lines 15 - 20, Update
the rebase instructions around the `develop` branch to explicitly select and
refresh the intended upstream base, including creating or resetting local
`develop` when absent or stale. Use a direct command referencing the chosen
remote branch (`origin/develop` or `origin/main`), then ensure the final `git
rebase develop` operates on that updated base.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.claude/skills/biome-unification-ci/SKILL.md:
- Around line 64-66: 검증 체크리스트에 저장소 루트 타입 검사를 위한 별도 `pnpm typecheck` 항목을 추가하세요.
기존 `pnpm ci:check`와 `pnpm build` 및 관련 앱 검사 항목은 유지해 푸시 전 필수 검증 절차가 모두 명시되도록 하세요.

In @.claude/skills/commit-push-pr/SKILL.md:
- Line 23: 커밋 예시를 저장소 규칙과 일치하도록 수정하세요. SKILL.md의 해당 실행 예시에서 `docs: ...` 앞에 문서
커밋용 이모지 접두사를 추가하고, 기존 커밋 타입과 제목 내용은 유지하세요.

---

Outside diff comments:
In @.claude/skills/rebase-issues-405-409/SKILL.md:
- Around line 44-53: Update the “완료 검증” guidance to avoid requiring a fixed `0
7` commit count. Validate that the working tree is clean and the rebased branch
has the expected ancestor relationship with `origin/main`; present `git rev-list
--left-right --count` only as a reference value, since the count may
legitimately change.

In @.claude/skills/univ-extends-bruno-sync/SKILL.md:
- Around line 48-51: 실패 대응 섹션의 BRUNO_REPO_URL 조건을 수정해 BRUNO_COLLECTION_DIR이 유효하게
설정된 경우에는 BRUNO_SOURCE_MODE=remote여도 URL 없이 동작할 수 있음을 반영하세요. 명시적 collection
directory가 없어서 원격 경로를 resolve해야 하는 경우에만 BRUNO_REPO_URL 미설정으로 즉시 실패한다고 문서화하고, 관련된
BRUNO_COLLECTION_PATH 조건은 기존 동작에 맞게 유지하세요.
- Around line 15-20: Update the rebase instructions around the `develop` branch
to explicitly select and refresh the intended upstream base, including creating
or resetting local `develop` when absent or stale. Use a direct command
referencing the chosen remote branch (`origin/develop` or `origin/main`), then
ensure the final `git rebase develop` operates on that updated base.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e58f68d9-666c-4df6-a997-74850fff652b

📥 Commits

Reviewing files that changed from the base of the PR and between a19ad86 and 4dd3f32.

📒 Files selected for processing (9)

.claude/skills/biome-unification-ci/SKILL.md
.claude/skills/bruno-codegen-react-query/SKILL.md
.claude/skills/commit-push-pr/SKILL.md
.claude/skills/rebase-issues-405-409/SKILL.md
.claude/skills/univ-extends-bruno-sync/SKILL.md
.claude/skills/university-web-rewrite-caution/SKILL.md
AGENTS.md
CLAUDE.md
llms.txt

- univ-extends-bruno-sync: sync-bruno.mjs 실제 우선순위대로 BRUNO_COLLECTION_DIR이 BRUNO_SOURCE_MODE보다 우선한다는 점을 반영하고, 존재하지 않는 develop 브랜치를 임의로 만들지 않고 origin/develop 존재 여부를 먼저 확인하도록 리베이스 절차 수정 - rebase-issues-405-409: 완료 검증에서 하드코딩된 커밋 개수(0 7) 기대값을 제거하고 조상 관계 + clean 상태 기준으로 검증하도록 수정 - biome-unification-ci: 검증 체크리스트에 누락되어 있던 루트 pnpm typecheck 추가 - commit-push-pr: 실행 예시 커밋 메시지에 저장소 규칙(이모지 접두사) 반영, CLAUDE.md 대신 AGENTS.md를 참조하도록 수정

coderabbitai

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.claude/skills/commit-push-pr/SKILL.md:
- Around line 26-31: Update the Execution Steps in the commit-push-PR skill
documentation to run pnpm typecheck and the relevant checks for changed
applications before git add. Keep the existing commit and push commands
unchanged, and explicitly require these validations before proceeding with the
push.

In @.claude/skills/univ-extends-bruno-sync/SKILL.md:
- Line 19: Update the develop-branch synchronization instructions in the rebase
workflow to remove the destructive git reset --hard behavior. Require a clean
working tree and a backup before any potentially destructive operation, or
replace it with a safe fast-forward-only update such as git pull --ff-only
origin develop while preserving the existing branch-creation path.
- Around line 18-23: 리베이스 실행 전에 현재 작업 브랜치로 복귀하는 단계를 추가하세요. `develop` 동기화 과정에서
브랜치를 전환하는 흐름을 수정해 `git rebase develop` 또는 `git rebase origin/main`이 작업 브랜치에서
실행되도록 하며, `feat/univ-extends` 같은 현재 브랜치가 `develop` 자체를 리베이스하는 no-op이 되지 않게 하세요.

In `@AGENTS.md`:
- Around line 41-42: AGENTS.md의 커밋 메시지 및 PR 제목/본문 작성 규칙에서 “기본적으로”라는 예외 표현을 제거하고,
해당 문장을 반드시 한국어로 작성하도록 통일하세요. 기존 규칙과 중복되면 한 곳만 남겨 규칙 간 충돌이 없도록 정리하세요.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bfe0745f-9a73-4272-832f-b744a0d49253

📥 Commits

Reviewing files that changed from the base of the PR and between 4dd3f32 and d6cc73d.

📒 Files selected for processing (7)

.claude/skills/biome-unification-ci/SKILL.md
.claude/skills/commit-push-pr/SKILL.md
.claude/skills/rebase-issues-405-409/SKILL.md
.claude/skills/univ-extends-bruno-sync/SKILL.md
.claude/skills/university-web-rewrite-caution/SKILL.md
AGENTS.md
CLAUDE.md

🚧 Files skipped from review as they are similar to previous changes (2)

.claude/skills/biome-unification-ci/SKILL.md
.claude/skills/rebase-issues-405-409/SKILL.md

coderabbitai · 2026-07-28T01:28:34Z

 ## Execution Steps

 ```bash
 git add <files>
-git commit -m "docs: 커밋/푸시/PR 서브에이전트 스킬 문서 추가"
+git commit -m "📝 docs: 커밋/푸시/PR 서브에이전트 스킬 문서 추가"
 git push -u origin <branch>


📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win

1. 푸시 전 필수 검사를 실행하도록 절차를 보강해 주세요.

현재 실행 단계에는 브랜치·상태 확인과 커밋/푸시만 있고, pnpm typecheck 및 변경된 앱의 관련 검사가 없습니다. 이 스킬을 따르는 에이전트가 검증 없이 푸시하지 않도록 git add 전에 해당 검사 단계를 명시해 주세요.

As per coding guidelines, 푸시 전에 pnpm typecheck와 관련 앱 검사를 실행해야 합니다.

제안된 수정

## Execution Steps

+pnpm typecheck
+# 변경된 앱의 관련 검사 실행
+
git add

</details> <details> <summary>🤖 Prompt for AI Agents</summary>

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.claude/skills/commit-push-pr/SKILL.md around lines 26 - 31, Update the
Execution Steps in the commit-push-PR skill documentation to run pnpm typecheck
and the relevant checks for changed applications before git add. Keep the
existing commit and push commands unchanged, and explicitly require these
validations before proceeding with the push.

</details>    _Source: Coding guidelines_ 

coderabbitai · 2026-07-28T01:28:34Z

+   - `git ls-remote --heads origin develop`으로 원격 `develop` 존재 여부를 먼저 확인한다.
+   - 존재하면: 로컬 `develop`을 최신 `origin/develop`으로 맞춘 뒤(`git checkout develop && git reset --hard origin/develop`, 로컬에 없으면 `git checkout -b develop origin/develop`) 그 브랜치로 리베이스한다.
+   - 존재하지 않으면(이 저장소는 기본적으로 `develop`이 없다): `origin/main`을 기준으로 직접 리베이스한다. 로컬 `develop`을 만들어서 우회하지 않는다.
+3. 리베이스 실행
+   - `origin/develop`이 있는 경우: `git rebase develop`
+   - 없는 경우: `git rebase origin/main`


🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

1. 리베이스 전에 작업 브랜치로 돌아오는 단계를 추가해야 합니다.

Line 19에서 develop으로 전환한 뒤 Line 22에서 바로 git rebase develop을 실행하므로, 현재 브랜치가 develop인 상태에서 자기 자신을 리베이스하는 no-op이 됩니다. 결과적으로 feat/univ-extends는 리베이스되지 않습니다.

수정 예시

- 존재하면: 로컬 `develop`을 최신 `origin/develop`으로 맞춘다. + - 동기화가 끝나면 `git checkout feat/univ-extends`로 작업 브랜치로 돌아간다. - `origin/develop`이 있는 경우: `git rebase develop`

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

- `git ls-remote --heads origin develop`으로 원격 `develop` 존재 여부를 먼저 확인한다.

- 존재하면: 로컬 `develop`을 최신 `origin/develop`으로 맞춘 뒤(`git checkout develop && git reset --hard origin/develop`, 로컬에 없으면 `git checkout -b develop origin/develop`) 그 브랜치로 리베이스한다.

- 존재하지 않으면(이 저장소는 기본적으로 `develop`이 없다): `origin/main`을 기준으로 직접 리베이스한다. 로컬 `develop`을 만들어서 우회하지 않는다.

3. 리베이스 실행

- `origin/develop`이 있는 경우: `git rebase develop`

- 없는 경우: `git rebase origin/main`

- `git ls-remote --heads origin develop`으로 원격 `develop` 존재 여부를 먼저 확인한다.

- 존재하면: 로컬 `develop`을 최신 `origin/develop`으로 맞춘 뒤(`git checkout develop && git reset --hard origin/develop`, 로컬에 없으면 `git checkout -b develop origin/develop`) 동기화가 끝나면 `git checkout feat/univ-extends`로 작업 브랜치로 돌아간다.

- 존재하지 않으면(이 저장소는 기본적으로 `develop`이 없다): `origin/main`을 기준으로 직접 리베이스한다. 로컬 `develop`을 만들어서 우회하지 않는다.

3. 리베이스 실행

- `origin/develop`이 있는 경우: `git rebase develop`

- 없는 경우: `git rebase origin/main`

🧰 Tools

🪛 SkillSpector (2.3.11)

[error] 19: [TM1] Tool Parameter Abuse: Tool parameters are crafted to achieve unintended or unsafe behavior. Parameter abuse can bypass intended safety checks (e.g. shell=True, --force, dangerous glob patterns).

Remediation: Validate all tool parameters against an allowlist. Reject dangerous parameter values (shell=True, --force, -rf /) and use safe defaults.

(Tool Misuse (TM1))

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.claude/skills/univ-extends-bruno-sync/SKILL.md around lines 18 - 23, 리베이스 실행 전에 현재 작업 브랜치로 복귀하는 단계를 추가하세요. `develop` 동기화 과정에서 브랜치를 전환하는 흐름을 수정해 `git rebase develop` 또는 `git rebase origin/main`이 작업 브랜치에서 실행되도록 하며, `feat/univ-extends` 같은 현재 브랜치가 `develop` 자체를 리베이스하는 no-op이 되지 않게 하세요.

coderabbitai · 2026-07-28T01:28:35Z

+   - `git fetch --all --prune`
+2. 기준 브랜치 확인
+   - `git ls-remote --heads origin develop`으로 원격 `develop` 존재 여부를 먼저 확인한다.
+   - 존재하면: 로컬 `develop`을 최신 `origin/develop`으로 맞춘 뒤(`git checkout develop && git reset --hard origin/develop`, 로컬에 없으면 `git checkout -b develop origin/develop`) 그 브랜치로 리베이스한다.


🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

2. git reset --hard로 로컬 작업을 삭제하지 않도록 안전장치를 추가해야 합니다.

Line 19의 명령은 추적 중인 미커밋 변경사항뿐 아니라 원격에 없는 로컬 develop 커밋도 참조에서 제거할 수 있습니다. 먼저 깨끗한 작업 트리와 백업을 요구하거나, 기본 동작을 git pull --ff-only origin develop처럼 안전하게 실패하는 방식으로 바꿔 주세요.

🧰 Tools

🪛 SkillSpector (2.3.11)

[error] 19: [TM1] Tool Parameter Abuse: Tool parameters are crafted to achieve unintended or unsafe behavior. Parameter abuse can bypass intended safety checks (e.g. shell=True, --force, dangerous glob patterns).

Remediation: Validate all tool parameters against an allowlist. Reject dangerous parameter values (shell=True, --force, -rf /) and use safe defaults.

(Tool Misuse (TM1))

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.claude/skills/univ-extends-bruno-sync/SKILL.md at line 19, Update the develop-branch synchronization instructions in the rebase workflow to remove the destructive git reset --hard behavior. Require a clean working tree and a backup before any potentially destructive operation, or replace it with a safe fast-forward-only update such as git pull --ff-only origin develop while preserving the existing branch-creation path.

Source: Linters/SAST tools

coderabbitai · 2026-07-28T01:28:35Z

+- Keep commit messages aligned with the Commit Message Convention below.
 - Write commit messages and PR title/body in Korean by default.


📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

1. 한국어 작성 규칙을 “반드시”로 통일하세요.

Line 8에서는 커밋 메시지와 PR 제목/본문을 반드시 한국어로 작성하도록 규정하지만, 이 구간은 “기본적으로”라고 표현해 예외를 허용하는 것처럼 보입니다. 두 규칙이 충돌하지 않도록 해당 문장을 커밋 메시지와 PR 제목/본문은 반드시 한국어로 작성한다.로 수정하거나 중복 문장을 제거해 주세요.

As per coding guidelines, 커밋 메시지와 PR 제목/본문은 반드시 한국어로 작성해야 합니다.

수정 예시

- - Write commit messages and PR title/body in Korean by default. + - Write commit messages and PR title/body in Korean.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

- Keep commit messages aligned with the Commit Message Convention below.

- Write commit messages and PR title/body in Korean by default.

- Keep commit messages aligned with the Commit Message Convention below.

- Write commit messages and PR title/body in Korean.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@AGENTS.md` around lines 41 - 42, AGENTS.md의 커밋 메시지 및 PR 제목/본문 작성 규칙에서 “기본적으로”라는 예외 표현을 제거하고, 해당 문장을 반드시 한국어로 작성하도록 통일하세요. 기존 규칙과 중복되면 한 곳만 남겨 규칙 간 충돌이 없도록 정리하세요.

Source: Coding guidelines

manNomi requested a review from wibaek as a code owner July 28, 2026 00:21

vercel Bot deployed to Preview – solid-connect-web-admin July 28, 2026 00:21 View deployment

vercel Bot deployed to Preview – solid-connection-web July 28, 2026 00:22 View deployment

chatgpt-codex-connector Bot reviewed Jul 28, 2026

View reviewed changes

vercel Bot deployed to Preview – solid-connect-university-web July 28, 2026 00:23 View deployment

vercel Bot deployed to Preview – solid-connect-web-admin July 28, 2026 00:26 View deployment

coderabbitai Bot reviewed Jul 28, 2026

View reviewed changes

Comment thread .claude/skills/biome-unification-ci/SKILL.md Outdated

Comment thread .claude/skills/commit-push-pr/SKILL.md Outdated

vercel Bot deployed to Preview – solid-connect-university-web July 28, 2026 00:27 View deployment

vercel Bot deployed to Preview – solid-connection-web July 28, 2026 00:28 View deployment

vercel Bot deployed to Preview – solid-connect-web-admin July 28, 2026 01:24 View deployment

vercel Bot deployed to Preview – solid-connection-web July 28, 2026 01:25 View deployment

vercel Bot deployed to Preview – solid-connect-university-web July 28, 2026 01:26 View deployment

coderabbitai Bot reviewed Jul 28, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

📝 스킬 문서를 .claude/skills로 승격하고 university-web 리라이트 주의사항 스킬 추가 - #611

📝 스킬 문서를 .claude/skills로 승격하고 university-web 리라이트 주의사항 스킬 추가#611
manNomi wants to merge 3 commits into
mainfrom
docs/ai-friendliness-improvements

manNomi commented Jul 28, 2026 •

edited

Loading

Uh oh!

vercel Bot commented Jul 28, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Jul 28, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

chatgpt-codex-connector Bot Jul 28, 2026

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jul 28, 2026

Uh oh!

coderabbitai Bot Jul 28, 2026

Uh oh!

coderabbitai Bot Jul 28, 2026

Uh oh!

coderabbitai Bot Jul 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		- Keep commit messages aligned with the Commit Message Convention below.
		- Write commit messages and PR title/body in Korean by default.

Uh oh!

Conversation

manNomi commented Jul 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

관련 이슈

작업 내용

특이 사항

리뷰 요구사항 (선택)

Uh oh!

vercel Bot commented Jul 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Jul 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector Bot Jul 28, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 28, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 28, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 28, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 28, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

manNomi commented Jul 28, 2026 •

edited

Loading

vercel Bot commented Jul 28, 2026 •

edited

Loading

coderabbitai Bot commented Jul 28, 2026 •

edited

Loading