Skip to content

improvement(docs): lib-bootstrap pattern + development branch model + jsDelivr URLs#68

Closed
Gumbees wants to merge 1 commit into
developmentfrom
improvement/lib-bootstrap-and-development-branch
Closed

improvement(docs): lib-bootstrap pattern + development branch model + jsDelivr URLs#68
Gumbees wants to merge 1 commit into
developmentfrom
improvement/lib-bootstrap-and-development-branch

Conversation

@Gumbees

@Gumbees Gumbees commented May 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Three load-bearing doc changes plus the repo-shape flip that the docs describe:

  • Lib bootstrap pattern. Leaf scripts that need shared helpers (oem-shared/lib/*, s3-api-lib/*) fetch them at runtime from jsDelivr pinned to a release tag and verify SHA256 before dot-sourcing. New commented-out block at the top of script-template-powershell.ps1, full recipe in CLAUDE.md under Common Patterns. Future-state note that CI may concatenate libs into fat scripts and turn the bootstrap into a no-op (Phase 2); until then, runtime fetch is the pattern.
  • Branching model flip. development is now the default branch ... all bug/, refactor/, improvement/, feature/ PRs target it (squash merge). main is production ... only development -> main promotions and hotfixes land here, always via merge commit. Hotfixes off main get back-merged into development so the next promotion doesn't undo them. Matches the Change Taxonomy merge-method canon. CLAUDE.md Git Workflow section rewritten end-to-end.
  • jsDelivr URL convention. Canonical script delivery URL is now https://cdn.jsdelivr.net/gh/dtc-inc/msp-script-library@<tag>/<path>. @release is the moving production tag (advanced on each promotion), @vX.Y.Z is the immutable pin. README gains a "Script Delivery (jsDelivr)" section explaining the tag conventions.

Repo operations also landed alongside this PR

The doc changes describe a model that already needed the repo to be reshaped. Done out-of-band before this PR opened so the PR body is accurate:

Test plan

  • PowerShell parser check on the modified template (passed locally with no errors)
  • Manual review: lib-bootstrap example block parses cleanly when uncommented (verified locally)
  • CLAUDE.md renders correctly on GitHub web view
  • Spot-check one re-targeted PR (mergeable + correct base branch)
  • Confirm jsDelivr URL serves expected content at @release once a real lib lands behind it
  • After this PR merges into development, dry-run the development -> main promotion flow (merge commit, then advance release tag)

@Gumbees Gumbees added type:enhancement New work that adds or improves capability (Halo: Enhancement) category:improvement Hardening or polish on existing capability labels May 11, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@Gumbees

Gumbees commented May 11, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favor of the OEM redo design captured at #74. The original approach (runtime lib bootstrap via jsDelivr, "baseline" naming with static policy files, hash placeholders) is being replaced with CI-generated fat scripts, internet check only at the install step, and RMM-env-var-driven configure scripts. New PRs will be opened against the foundation PR (separate, coming next).

Branch retained ... cherry-pick anything worth preserving from the new PR's branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:improvement Hardening or polish on existing capability type:enhancement New work that adds or improves capability (Halo: Enhancement)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant