Extract Types/Ids/SerializeReferences references and rework READMEs - #146
Draft
VPDPersonal wants to merge 15 commits into
Draft
Extract Types/Ids/SerializeReferences references and rework READMEs#146VPDPersonal wants to merge 15 commits into
VPDPersonal wants to merge 15 commits into
Conversation
- add self-contained EN/RU references for the Serializable Type System, ID System and SerializeReference Selector, extracted from the READMEs - drop the "— full reference" title suffix across all reference docs - add picker media: TypeSelectorDisplay/Required screenshots, an open-generic picking GIF, re-shot SerializableType/ComponentTypeSelector/window shots Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SuJE65oWmemhK78zC7NfUq
- rewrite the intro and polish the TOC blurbs - mirror the Serializable Type System section against Types.md: full TypeSelectorWindow feature list, Script-row note, generic-picking GIF - replace the TypeSelectorAttribute mechanics paragraph with reference links; drop sample callouts Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SuJE65oWmemhK78zC7NfUq
- Game.Combat harness (Weapons, Loadout, CombatModifiers, WeaponMount, GenericEffects) backing the Types.md screenshots and the open-generic GIF - update the sync-readmes skill Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SuJE65oWmemhK78zC7NfUq
VPDPersonal
added
type: documentation
- fix README count from fourteen/10-sample to sixteen/12-sample (SerializeReferences sample folder was missing) - drop stale StringIdRegistry mention — the type no longer exists in the codebase Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jf8Kdz9hobr77ckjYddJRn
…dates - explain in the TypeSelectorWindow section (EN + RU) that the generic argument page only lists types Unity can serialize as a field value (primitives, enum, string, UnityEngine.Object refs, [Serializable] classes/structs) Co-Authored-By: Claude <noreply@anthropic.com>
…overhaul # Conflicts: # Aspid.FastTools/Packages/tech.aspid.fasttools/Documentation/EN/README.md # Aspid.FastTools/Packages/tech.aspid.fasttools/Documentation/RU/README.md # README.md # README_RU.md
- reword the [SerializeReference] example comment: picking a type creates an instance and assigns it to the field (no misleading "stores the type") - explain Required = true inline: inspector warning plus build/CI-gate violation - applied to all six synced copies (root/Documentation READMEs and Types.md, EN/RU) Co-Authored-By: Claude <noreply@anthropic.com>
- add "Inspector type dropdown" media: picker GIF, missing-type repair and shared-reference Make unique notices - add "Bulk repair tabs" screenshots: Asset References and Project References - applied to both EN and RU copies Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
- drop the Generators solution-structure tree and TypeData/IdStructData reference copies (ls and the sources show them) - replace the Ids directory tree with a two-sentence layout summary keeping the non-obvious facts Co-Authored-By: Claude <noreply@anthropic.com>
…ReferenceTooling.md - move Bulk repair tabs, Project settings & build/CI gate and Headless CI sections out of SerializeReferences.md into a new SerializeReferenceTooling.md (EN/RU) with .meta files - SerializeReferences.md now covers only the Inspector side; intro, section list and Required/gate links repointed to the new doc - Types.md Required notice now links the fix instructions in the new doc; all 4 README footers link both references - register the new doc in the sync-readmes skill's per-feature list Co-Authored-By: Claude <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
1 participant
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.
Summary
Types.md,Ids.md,SerializeReferences.md, plusSerializeReferenceTooling.mdfor the project-wide side (bulk-repair tabs, project settings, build/CI gate, headless CI); drop the "— full reference" title suffix across all reference docsTypes.md, sample callouts removedTypeSelectorDisplay/Requiredscreenshots and an open-generic picking GIF; re-shotSerializableType,ComponentTypeSelectorand the picker windowDocsMediaharness backing the shots:Game.Combatdemo types (incl.GenericEffectsfor the open-generic GIF) and the SerializeReferences prefab set (WeaponPreset,SharedWeaponPreset,BrokenWeaponPreset)sync-readmesskill updated (sixteen READMEs, new per-feature list, staleStringIdRegistrynote dropped); derivable trees trimmed out of the Generators / IdsCLAUDE.mdNotes for review
[Serializable]candidates — the demo status effects carry the attribute for exactly that reasonTypes.md(intro, full feature list, media), so future edits should land inTypes.mdfirst and be mirroredmainis merged in a few times (the analyzer submodule → subtree move landed there mid-branch); the diff againstmainitself stays docs-only🇷🇺 Описание на русском
Кратко
Types.md,Ids.md,SerializeReferences.md, плюсSerializeReferenceTooling.mdпод проектную часть (вкладки массового ремонта, настройки проекта, build/CI-гейт, headless CI); у всех справочников снят суффикс «— full reference»Types.mdодин в один, колауты сэмплов убраныTypeSelectorDisplay/Requiredи гифка выбора открытого generic; переcнятыSerializableType,ComponentTypeSelectorи окно пикераDocsMediaв DevTests под съёмку: демо-типыGame.Combat(включаяGenericEffectsдля generic-гифки) и набор префабов SerializeReferences (WeaponPreset,SharedWeaponPreset,BrokenWeaponPreset)sync-readmes(шестнадцать README, новый список справочников, убрано устаревшее упоминаниеStringIdRegistry); изCLAUDE.mdGenerators и Ids вычищено выводимое из кодаЗаметки для ревью
[Serializable]-кандидатов — демо-статус-эффекты несут атрибут именно поэтомуTypes.md: правки сначала вTypes.md, затем зеркалитьmainвливался в ветку несколько раз (в него по ходу дела приехал перевод анализатора из сабмодуля в subtree); сам дифф относительноmainостаётся docs-only