Skip to content

🚫 [Reference Only]: addon-speed-analysis preserved snapshot (do not merge) - #125

Draft
Marius Storhaug (MariusStorhaug) wants to merge 6 commits into
mainfrom
addon-speed-analysis
Draft

🚫 [Reference Only]: addon-speed-analysis preserved snapshot (do not merge)#125
Marius Storhaug (MariusStorhaug) wants to merge 6 commits into
mainfrom
addon-speed-analysis

Conversation

@MariusStorhaug

Copy link
Copy Markdown
Member

Context vault creation, context storage, retrieval, and repeated session use complete faster while retaining encrypted storage and existing command behavior.

Changed: Common operations scale better with populated vaults

Exact-ID reads and writes no longer repeatedly scan every context file in a vault. Encryption setup and recursive object conversion reuse safe work where possible, reducing latency as vaults grow.

Changed: New PowerShell sessions load Context faster

Sodium is validated and loaded only when encryption or decryption is first needed, reducing startup overhead for sessions that import Context before using stored data.

Fixed: Context file operations stay inside the selected vault

Vault names containing path separators are rejected, and file operations use the actual discovered metadata file path instead of trusting a stored path value.

Technical Details

  • Adds per-session context-file and vault-key caches with invalidation on writes, removals, and vault deletion.
  • Moves hot-path collection, string, and file work to direct .NET APIs.
  • Uses Sodium 2.2.4 and validates cryptography commands lazily.
  • Migrates tests to Pester 6 and adds heavy workload and path-hardening coverage.
  • Aligns CI with Process-PSModule v6.1.13 and adds .github/zensical.toml.

No issue is linked; this PR was explicitly requested during the performance investigation.

Replace pipeline-heavy recursion with direct .NET collection and string operations while preserving secure-string and nested-object behavior.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Cache exact-ID file lookups and vault keypairs, defer Sodium loading until cryptography is used, and harden paths used for vault file operations.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Run the suite on Pester 6 and cover populated-vault workloads, repeated context roundtrips, cross-vault exact-ID results, and path traversal protections.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Wrap long calls, declare recursive output types, and document why in-memory cache helpers do not use ShouldProcess.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
- Pin workflow to v6.1.13 (SHA fb1bdb8fefd243292f779d2a856a38db6fe6daf4)
- Replace secrets: inherit with explicit APIKey secret per v6.0.0 breaking change
- Context has no test secrets so TestData is omitted

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Translates all mkdocs.yml settings to Zensical TOML format:
- Preserves palette colors (black/green dark, indigo/green light)
- Updates palette toggle icons from material/ to lucide/
- Retains all navigation features and markdown extensions
- Adds content.code.copy and pymdownx.superfences (Zensical standard)
- Social links and cookie consent carried over

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@MariusStorhaug Marius Storhaug (MariusStorhaug) changed the title 🩹 [Patch]: Common context operations complete faster 🚫 [Reference Only]: addon-speed-analysis preserved snapshot (do not merge) Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant