Skip to content

fix(editor): normalize documentation and harden editor utilities - #182

Merged
VPDPersonal merged 1 commit into
mainfrom
codex/editor-docs-audit
Sep 8, 2026
Merged

fix(editor): normalize documentation and harden editor utilities#182
VPDPersonal merged 1 commit into
mainfrom
codex/editor-docs-audit

Conversation

@VPDPersonal

Copy link
Copy Markdown
Owner

Summary

  • Normalize Editor XML documentation for public/protected API, remove XML from internal/private declarations, and keep ordinary C#/USS/UXML comments only where they explain non-obvious behavior.
  • Fix generic constraint filtering (Nullable<T> under struct, abstract classes under new()), validate reflected members before reading them, and copy assigned excluded-folder arrays so external mutation cannot bypass persistence and notifications.
  • Block keyboard adjustments/deletion of hidden navigation targets, guard invalid dot-grid dimensions that could hang the Editor, and clear inspector-header hover accents after its status changes to None.

Notes for review

Most of the 201 changed files contain documentation/comment cleanup. Behavior fixes include focused regression coverage; the invalid dot-spacing case is guarded in code without a live regression test that would hang the old implementation.

Validation: all 398 EditMode tests passed in Unity 6000.4. Roslyn XML/cref and documented-parameter checks passed across 212 C# files; git diff --check passed. USS/UXML content excluding comments and whitespace is unchanged.

🇷🇺 Кратко
  • XML-документация Editor приведена к правилам публичного API; XML у внутренних типов и непубличных членов удалён. Обычные комментарии оставлены для неочевидного поведения.
  • Исправлены generic-ограничения, проверка членов перед чтением через reflection и копирование массива исключённых папок.
  • Исправлены клавиатурные действия над скрытыми элементами, зависание сетки точек при некорректных параметрах и сброс подсветки заголовка после смены статуса.

Restrict XML documentation to public and protected API and remove redundant comments.

Fix generic constraints, reflection member validation, settings array ownership, hidden navigation actions, dot rendering, and hover reset.

Validation: 398 EditMode tests passed; XML documentation and diff checks passed.
@VPDPersonal VPDPersonal added type: fix Bug fix status: work-in-progress Draft / not ready for review area: editor Editor-only code labels Sep 8, 2026
@VPDPersonal
VPDPersonal marked this pull request as ready for review September 8, 2026 06:57
@github-actions github-actions Bot added status: needs-review Ready for review and removed status: work-in-progress Draft / not ready for review labels Sep 8, 2026
@VPDPersonal
VPDPersonal merged commit 0c383a0 into main Sep 8, 2026
3 of 5 checks passed
@VPDPersonal
VPDPersonal deleted the codex/editor-docs-audit branch September 8, 2026 07:02
@github-actions github-actions Bot removed the status: needs-review Ready for review label Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: editor Editor-only code type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant