Extract Types/Ids/SerializeReferences references and rework READMEs#146
Draft
VPDPersonal wants to merge 13 commits into
Draft
Extract Types/Ids/SerializeReferences references and rework READMEs#146VPDPersonal wants to merge 13 commits into
VPDPersonal wants to merge 13 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>
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— out of the oversized READMEs; drop the "— full reference" title suffix across all reference docsTypes.md, sample callouts removedTypeSelectorDisplay/Requiredscreenshots and an open-generic picking GIF; re-shotSerializableType/ComponentTypeSelector/ picker-window shotsDocsMediaharness (Game.Combatdemo types) backing the screenshots, includingGenericEffectsfor the open-generic GIF;sync-readmesskill updatedNotes 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 mirrored🇷🇺 Описание на русском
Кратко
Types.md,Ids.md,SerializeReferences.md(EN/RU) — вынесены из разросшихся README; у всех справочников снят суффикс «— full reference»Types.mdодин в один, колауты сэмплов убраныTypeSelectorDisplay/Requiredи гифка выбора открытого generic; переcнятыSerializableType/ComponentTypeSelector/ окно пикераDocsMediaв DevTests (демо-типыGame.Combat) для съёмки скриншотов, включаяGenericEffectsдля generic-гифки; обновлён скиллsync-readmesЗаметки для ревью
[Serializable]-кандидатов — демо-статус-эффекты несут атрибут именно поэтомуTypes.md: правки сначала вTypes.md, затем зеркалить