Skip to content

[Feat] Support bulk pagination defaults in overlays#41

Merged
samzong merged 1 commit into
mainfrom
feat/bulk-pagination-defaults
May 26, 2026
Merged

[Feat] Support bulk pagination defaults in overlays#41
samzong merged 1 commit into
mainfrom
feat/bulk-pagination-defaults

Conversation

@samzong

@samzong samzong commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add module-level overlay pagination defaults with command glob matching.
  • Merge bulk defaults before per-command parameter overrides so explicit command defaults still win.
  • Document the precedence and example schema.

Closes #34

Verification

  • make check
  • go test ./internal/codegen/render ./internal/lathecmd
  • git diff --check

Compatibility

  • Generated CommandSpec defaults can now come from defaults.pagination overlays.
  • Catalog/runtime schema is unchanged; runtime still receives only merged CommandSpec values.
  • Existing per-command params.<name>.default overlays continue to work and take precedence.

Checklist

  • Tests or focused verification cover the changed surface.
  • User-facing behavior changes are documented.
  • Generated output under internal/generated/, .cache/, and ad-hoc skills/<cli-name>/ directories is not committed.
  • Commits are signed off when this is ready to merge.

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong samzong merged commit a9bb3c7 into main May 26, 2026
3 checks passed
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.

Support bulk pagination defaults in overlays

1 participant