Refelection improvements

.github/CODEOWNERS

@@ -0,0 +1 @@
+/.editorconfig @BionicCode

.github/instructions/codestyle.instructions.md

@@ -0,0 +1,35 @@
+# Code style guidance (Clean Code)
+
+When proposing code changes, prefer applying Clean Code principles while adhering to existing repository standards.
+
+## Priority order
+
+1. Follow the repository's `.editorconfig` and established local conventions.
+2. Prefer Clean Code principles where they improve clarity without causing unnecessary churn.
+
+## Clean Code preferences (when appropriate)
+
+- Favor clarity and maintainability over cleverness.
+- Keep methods small and focused.
+- Use meaningful names and consistent terminology.
+- Minimize side effects and hidden behavior.
+- Avoid unnecessary complexity and premature abstraction.
+
+## General code style
+
+- Use consistent indentation (4 spaces).
+- Place `using` directives at the top of the file.
+- Keep related code together and organize files logically.
+- Use regions to group related code blocks.
+- Follow naming conventions (e.g., PascalCase for public members, camelCase for private members).
+- Write XML documentation comments for public APIs.
+- Prefer expression-bodied members for simple properties and methods.
+- Use string interpolation over concatenation.
+- Use region name also for the closing endregion directive
+- Prefer string.Equals for string equality test with specified StringComparison
+- When generating a `object.GetHashCode()` override, define a `private readonly int _hashCode` field to store the hash code value and create a related `int ComputeHashCode()` method to compute the hash code value from the constructor of the current type. Goal: make hash code computation a one-time operation. 
+
+## Refactoring scope
+
+- Do not perform broad renames or large structural refactors unless explicitly requested.
+- If a large refactor would help, propose it as an option and explain the tradeoffs.

.github/instructions/review.instructions.md

@@ -0,0 +1,23 @@
+# Code review output requirements (call-tree coverage footer)
+
+When performing a code review, always include a **Call-tree coverage footer** at the end of the review.
+
+When reviewing, prefer a **Clean Code** lens (readability, naming, and complexity) **without** requesting changes that conflict with the repository’s `.editorconfig` or established local conventions. Avoid broad refactors unless explicitly requested.
+
+## Call-tree coverage footer format
+
+Include the following items:
+
+- **Traversal completeness:** `Complete` or `Incomplete`.
+- **Coverage depth:** the number of expanded call steps (e.g., `A -> B -> C` = depth 3).
+- **Expanded symbols:** list the expanded call chain(s) (method names).
+- **Stopped at (when incomplete):** the last method reviewed including `file path` and `line range`.
+- **Unexpanded callees / reasons:** list any calls not expanded and why (e.g., definition not provided, reflection, dynamic dispatch, delegate/event, too many branches).
+- **Files needed for complete coverage (when incomplete):** list of files and types needed to achieve complete coverage.
+- **Review trigger:** the trigger for the review (e.g., AGENTS.md, Codex web, automatic review, GitHub PR @codex, GitHub automation etc.).
+- **Notes:** any additional relevant information about the coverage.
+
+## Definition of "Complete"
+
+“Complete” means complete **with respect to the code that is available to the assistant in the current prompt/context**.  
+If additional files are required to increase coverage, the review must explicitly request them. But always give feedback which files and types are needed to complete coverage in form of a list.

.github/workflows/sync-editorconfig-on-push.yml

@@ -0,0 +1,102 @@
+name: Sync .editorconfig from canonical (PR-only)
+
+on:
+  push:
+    branches-ignore:
+      - "chore/sync-editorconfig" # bot PR branch
+  pull_request:
+  workflow_dispatch:
+  # Optional polling safety net (recommended for dormant repos):
+  # schedule:
+  #   - cron: "17 * * * *"  # hourly at :17
+
+permissions:
+  contents: write
+  pull-requests: write
+
+env:
+  CANONICAL_RAW_URL: https://raw.githubusercontent.com/BionicCode/bioniccode-code-style/refs/heads/main/BionicCode.CodeStyle/BionicCode.CodeStyle/.editorconfig
+  SYNC_BRANCH: chore/sync-editorconfig
+
+# Concurrency scoped per PR or per ref (prevents cross-branch cancellations)
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  verify-on-pr:
+    if: github.event_name == 'pull_request'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout PR head
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.head_ref }}
+
+      - name: Download canonical .editorconfig
+        run: |
+          set -euo pipefail
+          curl -fsSL "${CANONICAL_RAW_URL}" -o canonical.editorconfig
+
+      - name: Ensure exactly one tracked .editorconfig (strict)
+        run: |
+          set -euo pipefail
+          count=$(git ls-files '**/.editorconfig' | wc -l | tr -d ' ')
+          if [ "$count" -ne 1 ]; then
+            echo "::error::Expected exactly one tracked .editorconfig (repo root). Found: $count"
+            git ls-files '**/.editorconfig' || true
+            exit 1
+          fi
+
+      - name: Fail if branch .editorconfig is out of sync
+        run: |
+          set -euo pipefail
+          if cmp -s .editorconfig canonical.editorconfig; then
+            echo ".editorconfig is in sync with canonical."
+          else
+            echo "::error::.editorconfig is OUT OF SYNC with canonical. Merge the sync PR created by this workflow."
+            exit 1
+          fi
+
+  open-or-update-sync-pr:
+    # PR-only syncing: create/update a PR on default-branch pushes, or manual runs
+    if: >
+      github.event_name != 'pull_request' && github.actor != 'github-actions[bot]'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout default branch
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.repository.default_branch }}
+
+      - name: Download canonical .editorconfig
+        run: |
+          set -euo pipefail
+          curl -fsSL "${CANONICAL_RAW_URL}" -o canonical.editorconfig
+
+      - name: Detect changes
+        id: diff
+        run: |
+          set -euo pipefail
+          if [ -f .editorconfig ] && cmp -s .editorconfig canonical.editorconfig; then
+            echo "changed=false" >> "$GITHUB_OUTPUT"
+          else
+            mv canonical.editorconfig .editorconfig
+            echo "changed=true" >> "$GITHUB_OUTPUT"
+          fi
+
+      - name: Create or update PR
+        if: steps.diff.outputs.changed == 'true'
+        uses: peter-evans/create-pull-request@v7
+        with:
+          branch: ${{ env.SYNC_BRANCH }}
+          base: ${{ github.event.repository.default_branch }}
+          title: "chore: sync .editorconfig from canonical"
+          commit-message: "chore(sync): update .editorconfig from canonical"
+          body: |
+            Automated sync of `.editorconfig` from the canonical repository.
+
+            Source: BionicCode/bioniccode-code-style (main)
+          add-paths: |
+            .editorconfig
+          labels: automation

AGENTS.md

@@ -0,0 +1,30 @@
+If user prompt only contains "<review>", consider it an instructions placeholder for the [REVIEW MISSION (default)](#review-mission-default) section which serves as the real task input:
+
+# REVIEW MISSION (default)
+Goal: Perform a deep, call-tree-based review focused on correctness of reflection identity/caching.
+Default entry point(s):
+- Start at: public API of SymbolReflectionInfoCache and SymbolReflectionInfoCacheKey
+- Trace: Full call tree.
+
+If the user prompt specifies a different entry point or path, follow the user prompt (user prompt overrides AGENTS.md).
+
+# PRIMARY INVARIANTS (Definition of Done)
+I should be able to trust that:
+1) Indexer parameters normalize to a stable identity across retrieval paths
+2) Parameter cache keys are consistent and won’t split/duplicate entries depending on acquisition path
+3) TypeData caching makes the hot path reflection-free after warmup
+4) Explicit interface accessors are handled correctly (no name-heuristic pitfalls)
+
+# OUTPUT FORMAT I WANT
+- Organize by file headers.
+- Use filename + 1-based line references like [L123].
+- Findings tagged: [ERROR]/[BUG]/[SECURITY]/[PERF]/[DESIGN]/[API]/[DOCS]/[TEST]/[STYLE]/[RISK].
+- After findings: “Coverage / Call-tree traversal depth” (which paths you traced, and where you had to stop due to missing context).
+
+# REVIEW DEPTH RULES
+- Always include a short “Call-tree” section showing the traced chain (at least: method A -> method B -> method C).
+- If you cannot fully trace due to missing files/context, stop and explicitly say where you stopped.
+
+# OPTIONAL: COMMANDS / EXECUTION
+- Do not run build/tests unless explicitly requested by the user.
+- Prefer static reasoning + code inspection.

BionicCode.Net.code-workspace

@@ -0,0 +1,8 @@
+{
+	"folders": [
+		{
+			"path": "."
+		}
+	],
+	"settings": {}
+}

BionicCode.Net.slnx

@@ -0,0 +1,43 @@
+<Solution>
+  <Configurations>
+    <Platform Name="Any CPU" />
+    <Platform Name="ARM" />
+    <Platform Name="ARM64" />
+    <Platform Name="x64" />
+    <Platform Name="x86" />
+  </Configurations>
+  <Folder Name="/Controls/">
+    <Project Path="src/BionicCode.Controls.Net.Wpf/BionicCode.Controls.Net.Wpf.csproj">
+      <Build Solution="Debug|Any CPU" Project="false" />
+    </Project>
+  </Folder>
+  <Folder Name="/Test/">
+    <Project Path="tests/BionicCode.Controls.Net.Wpf.Ui.Test/BionicCode.Controls.Net.Wpf.Ui.Test.csproj">
+      <Build Solution="Debug|Any CPU" Project="false" />
+    </Project>
+    <Project Path="tests/BionicCode.Net.Framework.Wpf.BionicCharts.Ui.Test/BionicCode.Net.Framework.Wpf.BionicCharts.Ui.Test.csproj" Id="83fdac1f-7bcd-4a38-9b13-b778024c50b2">
+      <Build Solution="Debug|Any CPU" Project="false" />
+    </Project>
+    <Project Path="tests/BionicCode.Utilities.UnitTest.Net/BionicCode.Utilities.Net.UnitTest.csproj">
+      <Build Solution="Release|Any CPU" Project="false" />
+    </Project>
+  </Folder>
+  <Folder Name="/Utilities/">
+    <Project Path="src/BionicCode.Utilities.Net.Common/BionicCode.Utilities.Net.Common.csproj" />
+    <Project Path="src/BionicCode.Utilities.Net.Wpf/BionicCode.Utilities.Net.Wpf.csproj" />
+    <Project Path="src/BionicCode.Utilities.Net/BionicCode.Utilities.Net.csproj" />
+    <Project Path="src/BionicCode.Utilities.Reflection/BionicCode.Utilities.Net.Reflection.csproj" Id="7078924e-729e-4dd9-8d4a-3b360c507d15" />
+  <Project Path="src/BionicCode.Utilities.Net.Profiling/BionicCode.Utilities.Net.Profiling.csproj" Id="6e13f596-7ab1-4df7-b9b4-afb4bd4001b9">
+    <BuildDependency Project="src/BionicCode.Utilities.Net.Common/BionicCode.Utilities.Net.Common.csproj" />
+  </Project>
+    <Project Path="src/BionicCode.Utilities.Net.Common.Isolation.Net6/BionicCode.Utilities.Net.Common.Isolation.Net6.csproj" />
+  </Folder>
+  <Project Path="src/BionicCode.SIUnits.Core/BionicCode.SIUnits.Core.csproj" />
+  <Project Path="Documentation/Documentation.csproj">
+    <BuildDependency Project="src/BionicCode.Utilities.Net.Common/BionicCode.Utilities.Net.Common.csproj" />
+    <BuildDependency Project="src/BionicCode.Utilities.Net.Profiling/BionicCode.Utilities.Net.Profiling.csproj" />
+    <BuildDependency Project="src/BionicCode.Utilities.Net.Wpf/BionicCode.Utilities.Net.Wpf.csproj" />
+    <BuildDependency Project="src/BionicCode.Utilities.Net/BionicCode.Utilities.Net.csproj" />
+    <Build Solution="Debug|Any CPU" Project="false" />
+  </Project>
+</Solution>