Skip to content

fix(conversation-memory): include built dist files in git - #351

Merged
baleen37 merged 1 commit into
mainfrom
fix/conversation-memory-dist
Feb 3, 2026
Merged

fix(conversation-memory): include built dist files in git#351
baleen37 merged 1 commit into
mainfrom
fix/conversation-memory-dist

Conversation

@baleen37

@baleen37 baleen37 commented Feb 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • conversation-memory 플러그인이 마켓플레이스 설치 후 동작하지 않는 문제 해결
  • .gitignore에서 dist/ 전역 제외 규칙 제거
  • 빌드된 MCP 서버 및 CLI 파일을 git에 포함

Problem

conversation-memory 플러グ인이 설치되었지만 MCP 서버가 동작하지 않음:

  1. dist/ 디렉토리가 .gitignore로 인해 git에 포함되지 않음
  2. 마켓플레이스에 배포 시 dist 파일 누락
  3. 설치된 플러그인에 dist/mcp-server.mjs, dist/cli.mjs 없음
  4. MCP 서버 실행 불가

Changes

  • .gitignore: dist/ 전역 제외 규칙 제거 및 예외 처리 정리
  • plugins/conversation-memory/dist/cli.mjs: CLI 빌드 파일 추가
  • plugins/conversation-memory/dist/mcp-server.mjs: MCP 서버 빌드 파일 추가

Test plan

  • 로컬에서 npm run build 성공
  • MCP 서버 실행 확인: node dist/mcp-server.mjs
  • CLI 실행 확인: node dist/cli.mjs --help
  • dist 파일이 git에 포함됨 확인
  • 마켓플레이스 배포 후 플러그인 재설치하여 동작 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated repository configuration files to modify build artifact tracking behavior.

conversation-memory 플러그인이 동작하지 않는 문제 해결:
- dist/ 디렉토리가 .gitignore로 무시되어 마켓플레이스에 포함되지 않음
- .gitignore에 conversation-memory/dist/ 예외 추가
- 빌드된 MCP 서버 및 CLI 파일 커밋

이제 플러그인 설치 시 dist/mcp-server.mjs와 dist/cli.mjs가 포함됨

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@baleen37
baleen37 enabled auto-merge (squash) February 3, 2026 06:41
@coderabbitai

coderabbitai Bot commented Feb 3, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The .gitignore file was modified to remove the global ignore pattern for the dist/ directory and the whitelist exception for plugins/suggest-compacting/dist/, allowing these directories to be tracked by version control instead of being excluded.

Changes

Cohort / File(s) Summary
Gitignore Configuration
.gitignore
Removed global dist/ directory ignore rule and the whitelist exception for plugins/suggest-compacting/dist/; directories now follow remaining ignore rules.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Three lines deleted from ignore's hold,
dist/ paths now permitted, bold and free,
Version control shall track them all,
No longer hidden in the shadows tall! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/conversation-memory-dist

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 and usage tips.

@baleen37
baleen37 merged commit cb51aaa into main Feb 3, 2026
1 of 2 checks passed
@baleen37
baleen37 deleted the fix/conversation-memory-dist branch February 3, 2026 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant