Skip to content

fix: 3 platform icin eklenti paketi, unix kurucu onarimi, clippy temizligi - #3

Merged
Ercaner1988 merged 6 commits into
mainfrom
fix/cross-platform-mcpb
Sep 1, 2026
Merged

fix: 3 platform icin eklenti paketi, unix kurucu onarimi, clippy temizligi#3
Ercaner1988 merged 6 commits into
mainfrom
fix/cross-platform-mcpb

Conversation

@Ercaner1988

Copy link
Copy Markdown
Owner

Meslektaslara ulasan kurulum yolunda iki gercek kirik.

1. .mcpb yalnizca Windows icin uretiliyordu

Manifest win32, darwin ve linux destekledigini iddia ediyor ama paketin icinde yalnizca claude-code-setup.exe vardi. Mac veya Linux'ta eklenti kurulur ama calismaz.

  • package-extension.py artik entry_point'i ve compatibility.platforms'u verilen binary'ye gore yazar
  • release.yml uc kosucuda da paketler: claude-code-setup-{windows,macos,linux}.mcpb
  • ubuntu-latest'te python komutu yok, actions/setup-python eklendi

2. install-macos.sh hic calismiyordu

GitHub API JSON'u "key": "value" biciminde donuyor — iki nokta sonrasi bosluk var. Betigin grep kaliplarinda bosluk yoktu, hicbir sey eslesmiyordu:

eski kalip '"tag_name":"[^"]*'   -> eslesme yok
yeni kalip '"tag_name": *"[^"]*' -> v0.1.5

Betik Could not fetch latest release ile cikiyordu. Ayrica macOS ikilisi sabit kodluydu ve dokumanlar ayni betigi Linux icin de gosteriyordu; Linux'ta Mach-O binary inip cannot execute binary file verirdi.

  • grep kaliplari bosluk toleransli
  • uname -s ile isletim sistemi tespiti, desteklenmeyen sistemde acik hata

3. Dokumanlar yanlis sey vaat ediyordu

Kurucularin "Claude Code extension'i register ettigi" yaziyordu — etmiyorlar, yalnizca CLI'i kuruyorlar. .mcpb yolu ise hic belgelenmemisti. README ve INSTALLATION.md'de iki kurulum yolu ayrildi.

Dogrulama

Paketleyici uc platform icin de dogru entry_point ve tek platformluk compatibility uretiyor. Varlik secme mantigi gercek release JSON'u uzerinde sinandi: Darwin -> macos ikilisi, Linux -> linux ikilisi, FreeBSD -> acik hata.

🤖 Generated with Claude Code

Ercaner1988 and others added 6 commits September 1, 2026 18:28
install-windows.ps1 varsayilan olarak %PROGRAMFILES% altina yaziyordu; bu
dizin yonetici yetkisi ister, normal kullanicida "erisim reddedildi" verir.
Meslektaslarin tek komutluk kurulumu bu yuzden sessizce dusuyordu.

- Hedef: %LOCALAPPDATA%\Programs\ClaudeCodeSetup (yetki gerekmiyor)
- Program Files icin gereken icacls adimi kaldirildi

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
CI'in "cargo clippy --all-targets -- -D warnings" adimi uc platformda da
duruyordu:
  error: struct `McpServer` is never constructed
  error: field `jsonrpc` is never read

Ikisi de hicbir yerde kullanilmiyordu. Bu olu kod once de vardi; fmt
hatasi giderilince clippy adimi calisabilir hale geldi ve bunlari buldu.

- McpServer isaretci struct'i silindi (hic kurulmuyordu)
- JsonRpcRequest.jsonrpc alani silindi (hic okunmuyordu; serde bilinmeyen
  alanlari zaten yok sayar, gelen istekler etkilenmiyor)

Dogrulama: cargo fmt --check, cargo clippy --all-targets -D warnings,
cargo test (30 gecti) ve canli MCP el sikismasi + tools/call.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
README ve INSTALLATION'in 7'ser dili neredeyse birebir ayni dugumleri
uretiyordu: 489 dugumun ~120'si ceviri yinelemesiydi. Kanonik olanlari
tutup gerisini .graphifyignore ile disladik.

489 -> 273 dugum, 33 -> 19 topluluk. Kalan topluluklar dosya sinirlarina
temiz oturuyor ve hepsi adlandirildi.

Not: .graphifyignore yalnizca sifirdan kurulumda etkili. graphify'in
artimli yeniden derlemesi degismeyen dosyalarin dugumlerini eski
graph.json'dan koruyor, bu yuzden ignore'a yeni desen eklenince
graph.json'un silinip yeniden kurulmasi gerekiyor.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Iki gercek kirik vardi; ikisi de meslektaslara ulasan yolda.

1) .mcpb yalnizca Windows icin uretiliyordu ama manifest win32+darwin+linux
   destekledigini iddia ediyordu. Mac veya Linux'ta eklenti kurulur, icinde
   yalnizca .exe oldugu icin calismazdi.
   - package-extension.py artik entry_point'i ve compatibility.platforms'u
     verilen binary'ye gore yazar; paket adina platform ekler
   - release.yml uc kosucuda da paketler: windows/macos/linux .mcpb

2) install-macos.sh hic calismiyordu. GitHub API JSON'u "key": "value"
   biciminde donuyor (iki nokta sonrasi bosluk); betigin grep kaliplarinda
   bosluk yoktu, hicbir sey eslesmiyor ve "Could not fetch latest release"
   ile cikiyordu. Ayrica macOS ikilisi sabit kodluydu, dokumanlar ayni
   betigi Linux icin de gosteriyordu -> Linux'ta Mach-O binary inip
   "cannot execute binary file" verirdi.
   - grep kaliplari bosluk toleransli
   - uname -s ile isletim sistemi tespiti; desteklenmeyen sistemde acik hata

Dokumanlar da yanlisti: kurucularin "Claude Code extension'i register
ettigi" yaziyordu, etmiyorlar. README ve INSTALLATION.md'de iki kurulum
yolu (eklenti / CLI) ayrildi, .mcpb yolu ilk kez belgelendi.

Dogrulama: paketleyici uc platform icin de dogru entry_point ve tek
platformluk compatibility uretiyor; varlik secme mantigi gercek release
JSON'u uzerinde Darwin/Linux/desteklenmeyen icin sinandi.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
graphify-out/ bir onceki commit'e yanlislikla girdi: .gitignore girdisi
diger dalda kalmisti, bu dal main'den ciktigi icin onu gormedi.

Uretilen artefakt (graph.json, graph.html, AST onbellegi) repoda durmamali.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Ercaner1988 Ercaner1988 changed the title fix: eklenti paketini 3 platform icin uret, unix kurucuyu onar fix: 3 platform icin eklenti paketi, unix kurucu onarimi, clippy temizligi Sep 1, 2026
@Ercaner1988
Ercaner1988 merged commit 6ad1a7c into main Sep 1, 2026
3 checks passed
@Ercaner1988
Ercaner1988 deleted the fix/cross-platform-mcpb branch September 1, 2026 16:25
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