ci+feat: paketi uc kosucuda calistir, macOS icin universal ikili - #7
Merged
Merged
Conversation
macOS ve Linux paketlerinin calisip calismadigini bugune kadar kimse dogrulamamisti; ikililer yalnizca derleniyordu. Bu, dogrulamayi bir gonullunun makinesine bagimli birakiyordu. Daha ciddisi CI kaydindan cikti: macOS kosucusu `macos-26-arm64` (host aarch64-apple-darwin), yayinlanan ikili ise x86_64-apple-darwin olarak capraz derleniyor ve hic calistirilmiyordu. Apple Silicon bir Mac'te bu ikili ancak Rosetta 2 ile calisir. - smoke-test-extension.py: paketi isletim sisteminin kendi unzip'iyle acar (calistirma izni ancak boyle sinanir), manifest'teki komutu calistirir, MCP el sikismasi + arac listesi + salt-okunur bir aracin gercek cagrisi. Ayrica sunucu surumu ile manifest surumunun ve ilan edilen araclar ile sunulanlarin esit oldugunu denetler. - release.yml artik her PR'da da kuru kosu yapiyor: ayni derleme, ayni paketleme, duman testi. Yukleme yalnizca v* etiketinde. Bozuk paket etiketlenmeden yakalanir. - rust.yml'deki yinelenen "Build release" adimi kaldirildi. Yerel dogrulama (Windows): saglam paket gecti; kasitli bozulmus uc paket (surum kaymasi, eksik arac bildirimi, eksik ikili) dogru mesajla ve cikis kodu 1 ile dustu. Calistirma izni dali Windows'ta sinanamaz, onu macOS/Linux kosuculari sinayacak. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Yayinlanan macOS ikilisi yalnizca x86_64'tu. Apple Silicon bir Mac'te bu ikili ancak Rosetta 2 ile calisir ve macOS Rosetta'yi varsayilan olarak kurmuyor. Bir onceki commit'in duman testi ARM kosucuda gecti, ama bu yalnizca kosucuda Rosetta oldugunu kanitliyordu; Rosetta'siz bir Mac icin hicbir sey soylemiyordu. - release.yml: macOS'ta x86_64-apple-darwin ve aarch64-apple-darwin ayri derlenip `lipo -create` ile birlestiriliyor. `lipo -archs` ile iki dilimin de varligi dogrulaniyor, eksikse is duser. Varlik adi: claude-code-setup-macos-universal (.mcpb adi degismedi). - Duman testi macOS'ta iki kez kosuyor: varsayilan kosu arm64 dilimini, SMOKE_ARCH=x86_64 kosusu Intel Mac'lerin alacagi dilimi calistiriyor. - smoke-test-extension.py: SMOKE_ARCH destegi (`arch -<mimari>`); macOS disinda verilirse acik hatayla reddediliyor. - install-macos.sh: once universal, bulamazsa x86_64. Geri dusus sart: bu dal birlesip v0.1.7 cikana kadar en son surumde universal yok, yalnizca universal arayan kurucu o aralikta Mac'te kirilirdi. - INSTALLATION.md: elle indirme satirinda ikili adi sabitlenmisti; kurucuya yonlendirildi. Kanca kurulumu git deposunda yapilacak sekilde ayrildi (ev dizininde zaten atlaniyordu). - Surum 0.1.7. Yerel dogrulama: kurucu secim mantigi gercek v0.1.6 JSON'unda x86_64'e, universal iceren surumde universal'e dustu; Linux ikisinde de dogru. Windows'ta duman testi 0.1.7 ile gecti; SMOKE_ARCH macOS disinda reddedildi. lipo ve arm64/x86_64 dilim kosulari yalnizca macOS kosucusunda sinanabilir. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Eklenti paketinin macOS ve Linux'ta calistigini bugune kadar kimse dogrulamamisti — ikililer yalnizca derleniyordu. Bu PR iki sey yapiyor: yayinlanacak paketi her PR'da uc sanal kosucuda gercekten calistiriyor, ve macOS icin universal ikili yayinliyor.
1. Duman testi (her PR'da)
release.ymlartik PR'larda kuru kosu yapiyor: ayni derleme, ayni paketleme, sonrasmoke-test-extension.py:unzip'iyle acar (calistirma izni ancak boyle sinanir; PythonzipfileUnix izinlerini uygulamaz)statusaracinin gercek cagrisiYukleme adimi yalnizca
v*etiketinde calisir. Yerelde kasitli bozulmus uc paket (surum kaymasi, eksik arac, eksik ikili) dogru mesajla dustu.2. macOS universal ikili
CI kaydi: macOS kosucusu
macos-26-arm64, yayinlanan ikili x86_64. Apple Silicon'da bu ancak Rosetta 2 ile calisir; macOS Rosetta'yi varsayilan olarak kurmuyor. Onceki duman kosusu ARM kosucuda gecti, ama bu yalnizca kosucuda Rosetta oldugunu kanitliyordu.lipo -createile birlestiriliyor;lipo -archsiki dilimi dogrulamazsa is duserSMOKE_ARCH=x86_64kosusu Intel Mac'lerin alacagi dilimi calistiririnstall-macos.shonce universal'i, bulamazsa x86_64'u arar — bu dal birlesip v0.1.7 cikana kadar en son surumde universal yok; geri dusus olmazsa kurucu o aralikta kirilirdi (iki durum da gercek release JSON'unda sinandi)INSTALLATION.md'deki elle indirme satiri ikili adini sabitliyordu, kurucuya yonlendirildiSinirlar
unzip'ini kullanir.statusaracini cagirir;memory_index'in gomu modeli yolu (yuzlerce MB indirme) CI'da kosmaz.contents: writetanimli (yukleme icin). Fork PR'larinda token zaten salt-okunura dusuyor; yukleme etiket disinda hic calismiyor.🤖 Generated with Claude Code