🩹 [Patch]: Common context operations complete faster#122
Draft
Marius Storhaug (MariusStorhaug) wants to merge 6 commits into
Draft
🩹 [Patch]: Common context operations complete faster#122Marius Storhaug (MariusStorhaug) wants to merge 6 commits into
Marius Storhaug (MariusStorhaug) wants to merge 6 commits into
Conversation
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>
Marius Storhaug (MariusStorhaug)
added
the
Patch
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
- 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>
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
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>
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.
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 also 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.
Changed: CI aligned with Process-PSModule v6.1.13
The reusable workflow is pinned to
v6.1.13(fromv5.4.6). Per thev6.0.0breaking change,secrets: inheritis replaced with an explicitAPIKeysecret. Context has no test-specific secrets soTestDatais omitted.Adopted release-note items:
TEST_*secrets removed; calling convention migrated to explicitAPIKey.Skipped / not applicable:
TestDataJSON secret (v6.0.0): Context has no test secrets, so theTestDatainput is intentionally omitted.ImportantFilePatterns(v5.5.0): Defaults (^src/,^README\.md$) match this repo's layout; no override needed.$IsWindowsshim removal (v5.4.7): internal framework change, no consumer action.Technical Details
No issue is linked; this PR was explicitly requested during the performance investigation.