Skip to content

feat(cache): support ALZ library refs and overwrite flag in cache create#244

Merged
matt-FFFFFF merged 4 commits intomainfrom
copilot/add-cache-create-features
Apr 27, 2026
Merged

feat(cache): support ALZ library refs and overwrite flag in cache create#244
matt-FFFFFF merged 4 commits intomainfrom
copilot/add-cache-create-features

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

  • Add --library-overwrite-enabled flag wired to AlzLib.Options.AllowOverwrite
  • Allow --library to accept ALZ-library references in the form path@ref
  • Allow --library to be specified multiple times
  • Update flag help text and command long description
  • Add unit tests for library reference parsing
  • Move parseLibraryReference into the alzlib package as public constructors
    • Add NewAlzLibraryReferenceFromString (inverse of AlzLibraryReference.String())
    • Add NewCustomLibraryReferenceFromString for symmetry
    • Add universal NewLibraryReference with auto-detection
  • Use alzlib.NewLibraryReference from the cache create command
  • Move parser tests to metadata_test.go and cover the new constructors

Copilot AI and others added 2 commits April 27, 2026 14:01
…o cache create

Agent-Logs-Url: https://github.com/Azure/alzlib/sessions/edcee89c-eb65-4920-964a-6d9121e6e3c8

Co-authored-by: matt-FFFFFF <16320656+matt-FFFFFF@users.noreply.github.com>
…ference

Agent-Logs-Url: https://github.com/Azure/alzlib/sessions/edcee89c-eb65-4920-964a-6d9121e6e3c8

Co-authored-by: matt-FFFFFF <16320656+matt-FFFFFF@users.noreply.github.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 27, 2026

Codecov Report

❌ Patch coverage is 44.82759% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.91%. Comparing base (8eb327c) to head (e3a34a9).

Files with missing lines Patch % Lines
cmd/alzlibtool/command/cache/create.go 0.00% 30 Missing ⚠️
metadata.go 92.85% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   47.76%   47.91%   +0.14%     
==========================================
  Files          54       54              
  Lines        4865     4913      +48     
==========================================
+ Hits         2324     2354      +30     
- Misses       2255     2272      +17     
- Partials      286      287       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matt-FFFFFF
Copy link
Copy Markdown
Member

@copilot fix this minor lint error

  Error: cmd/alzlibtool/command/cache/create.go:282:3: missing whitespace above this line (too many statements above if) (wsl_v5)
  		ref := s[idx+1:]
  		^
  1 issues:
  * wsl_v5: 1

…erence

Agent-Logs-Url: https://github.com/Azure/alzlib/sessions/2d6ab067-3b9c-4968-a532-e07bb61d5e03

Co-authored-by: matt-FFFFFF <16320656+matt-FFFFFF@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 27, 2026

@copilot fix this minor lint error

...

Fixed in c525cdb by adding a blank line between the local var declarations and the inner if in parseLibraryReference. Verified locally with golangci-lint run ./cmd/alzlibtool/command/cache/... — 0 issues.

…tection

Agent-Logs-Url: https://github.com/Azure/alzlib/sessions/6bceed06-ced3-40bc-8b0b-78635d478262

Co-authored-by: matt-FFFFFF <16320656+matt-FFFFFF@users.noreply.github.com>
@matt-FFFFFF matt-FFFFFF added this pull request to the merge queue Apr 27, 2026
Merged via the queue into main with commit 5f18137 Apr 27, 2026
8 checks passed
@matt-FFFFFF matt-FFFFFF deleted the copilot/add-cache-create-features branch April 27, 2026 14:49
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.

4 participants