Update v0.0.60 migration doc - #14
Conversation
Reviewer's GuideRefines the v0.0.60 migration documentation for the new sys-mappings component by expanding the domain configuration section, detailing required tooling script updates, and adding concrete migration steps and reference links to the vnext-example repo. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe v0.0.60 migration guide is expanded with a new "Domain projesi yapılandırması" subsection for Changesv0.0.60 Migration Guide — sys-mappings Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the migration documentation for v0.0.60, detailing how to configure domain projects and tooling scripts for the new mappings directory. The review feedback suggests improving Turkish grammar consistency in the introduction and resolving a potential ReferenceError in the validate.js documentation snippet by using a literal string instead of a configuration variable.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| ### Domain projesi yapılandırması | ||
|
|
||
| Domain bileşen ağacına `Mappings/` dizini eklendi: | ||
| Bu bölümdeki adımlar mevcut bir domain projesini v0.0.60'a taşırken atılır. Çalışan bir referans için [`vnext-example`](https://github.com/burgan-tech/vnext-example) reposuna bakın — aşağıdaki yapılandırmanın tamamı orada hazırdır. |
There was a problem hiding this comment.
Dil bilgisi ve anlatım uyumunu artırmak adına bu cümle daha doğal bir Türkçe ile yeniden yazılabilir. Etken fiil olan "taşırken" ile edilgen fiil olan "atılır" kullanımı anlatım bozukluğuna yol açmaktadır. Bunun yerine "taşınırken uygulanır" veya "taşınırken atılır" şeklinde edilgen uyumu sağlanabilir.
Öneri:
Bu bölümdeki adımlar mevcut bir domain projesi v0.0.60'a taşınırken uygulanır. Çalışan bir referans için [vnext-example](https://github.com/burgan-tech/vnext-example) reposuna bakın — aşağıdaki yapılandırmanın tamamı orada hazırdır.
| **`validate.js`** — dört noktada genişletilir: | ||
|
|
||
| ```js | ||
| // requiredExports ve componentGetters dizilerine: | ||
| 'getMappings', | ||
| // vnextDirs ve expectedDirs dizilerine: | ||
| pathsConfig.mappings, | ||
| // directoryToSchemaType eşlemesine (mapping → 'mapping' şeması): | ||
| [pathsConfig.mappings]: 'mapping', | ||
| ``` |
There was a problem hiding this comment.
Potansiyel Hata / İyileştirme Fırsatı:
validate.js dosyasında vnextDirs ve expectedDirs gibi diziler genellikle modül seviyesinde (global) tanımlanır. Eğer pathsConfig değişkeni bu dizilerin tanımlandığı kapsamda (scope) henüz yüklenmemiş veya tanımlanmamışsa, pathsConfig.mappings kullanımı ReferenceError hatasına yol açabilir.
test.js dosyasındaki örnekte olduğu gibi doğrudan 'Mappings' string değerini kullanmak veya pathsConfig'in ilgili kapsamda tanımlı olduğundan emin olmak daha güvenli bir yaklaşım olacaktır.
Örnek alternatif kullanım:
// vnextDirs ve expectedDirs dizilerine:
'Mappings',
Özet
Etkilenen Bölüm
docs/)architecture/)business/)product/)blog/)Dil
Local Doğrulama
npm run buildbaşarılınpm run startile gözle kontrol edildiİlgili Phase / Issue
Summary by Sourcery
Document mappings support in domain projects for the v0.0.60 migration, including configuration, tooling updates, and example references.
Documentation:
Summary by CodeRabbit