ci: add org project tracker scripts - #158
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Update all internal references to reflect the repository rename: - .trunk/trunk.yaml repo name - workflow repository guards (checksum, rclone-action, deploy-gh-pages) - win-install safe.directory path - docs/README.md VSCode and jsDelivr URLs
Deploying zi-src with
|
| Latest commit: |
3d9395e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4849dd5c.zi-src.pages.dev |
| Branch Preview URL: | https://ci-org-project-tracker-scrip.zi-src.pages.dev |
There was a problem hiding this comment.
Pull request overview
Adds organization-wide maintenance scripts intended to roll out consistent labels and a project-tracker workflow across z-shell repositories, along with several repo-identity/CI/config updates.
Changes:
- Add org maintenance scripts for label syncing and for opening per-repo PRs that add a project-tracker workflow.
- Update multiple repo identity references from
z-shell/zi-srctoz-shell/srcacross docs/CI/Trunk config. - Add vendored Zsh snippets (
revolver,color) and tighten installer/CI checks aroundzi.zshpresence.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/sync-labels.sh | New script to enforce a canonical label set across repos via gh label create --force. |
| scripts/add-project-workflow.sh | New script to clone repos, add project-tracker.yml, push a branch, and open PRs. |
| scripts/repos.txt | New repo list input used by the org maintenance scripts. |
| lib/zsh/snippets/revolver | Adds vendored spinner utility (currently contains CLI/runtime issues). |
| lib/zsh/snippets/color.zsh | Adds vendored color helper (shebang/invocation behavior needs adjustment). |
| lib/sh/install.sh | Tightens install success check to require zi.zsh exists after clone. |
| docs/README.md | Updates links to reflect z-shell/src naming. |
| .trunk/trunk.yaml | Updates Trunk repo identity from zi-src to src. |
| .github/workflows/win-install.yml | Updates safe.directory path and adds a smoke test verifying zi.zsh exists. |
| .github/workflows/rclone-action.yml | Gates job execution on z-shell/src. |
| .github/workflows/deploy-gh-pages.yml | Gates job execution on z-shell/src. |
| .github/workflows/checksum.yml | Adds concurrency config and gates job execution on z-shell/src. |
| .github/workflows/check-linux.yml | Adds a new drift check step for init.zsh via sync-init.sh. |
Comments suppressed due to low confidence (1)
lib/zsh/snippets/revolver:305
- The
--styleparsing logic clears the default style: when no--styleis provided,styleis set todots, thenshift styleruns and shifts the scalar away, leaving$styleempty and causing the spinner lookup to fail. Handle the default separately (e.g., only shift whenstylewas provided byzparseopts, then set the default after).
if [[ -z $style ]]; then
style='dots'
fi
if [[ -n $style ]]; then
shift style
ctx="$1"
fi
if [[ -z $_revolver_spinners[$style] ]]; then
echo $(color red "Spinner '$style' is not recognised")
exit 1
fi
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com>
Deploying src with
|
| Latest commit: |
c00fab3
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e4a082b9.zi-src.pages.dev |
| Branch Preview URL: | https://ci-org-project-tracker-scrip.zi-src.pages.dev |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com>
|
@copilot apply changes based on the comments in this thread |
… URLs Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com>
Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com>
Applied the linked review-thread fixes in commits Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
* chore: remove deprecated files and configs Remove obsolete cspell word lists, editorconfig, vscode settings, and retired workflows (rebase, stale, sync-labels, wrangler). Drop legacy installer scripts i.sh and install_alpha.sh. * ci: update workflows and trunk configuration - Bump actions/checkout to v6.0.2 and shellcheck to 2.0.0 - Add lib/zsh/** and tests/** to push/PR path triggers - Add permissions: contents: read and timeout-minutes: 10 - Use github.ref in concurrency group - Replace ~/.zi hard-coded paths with XDG_DATA_HOME equivalents - Add unit-fixture step calling tests/installers.sh - Remove wrangler deploy job; update trunk linter config * feat: add sync-init.sh and tests/installers.sh sync-init.sh verifies (and optionally replaces) the local lib/zsh/init.zsh against the canonical GitHub raw main copy. Supports --write, --local, --remote, --checksum-url and --no-checksum flags. tests/installers.sh provides a POSIX sh test suite covering: - script syntax validation (sh -n / zsh -n) - checksum integrity checks - loader install with XDG path and branch-override assertions - standalone zpmod delegation test - sync-init fixture round-trip test * refactor: migrate installer scripts to XDG paths and harden POSIX install.sh: - Add set -eu; use WORKDIR=$(mktemp -d) with safe trap cleanup - Replace ~/.zi with XDG_DATA_HOME-aware path - Fix sed command for ZI[STREAM] substitution - Correct cd fallback from return to exit 1 - Guard branch pull with BOPT; add exit 1 on failed clone - Fix .zshrc loader snippet variable quoting install_zpmod.sh: - Add set -eu, file header and WORKDIR trap - Add pure-POSIX _zi_ver_ge() replacing expr string comparison - Honour XDG_DATA_HOME for ZI_HOME default - Replace exit 255 with exit 1 throughout lib/zsh/init.zsh: - Update default branch to use ZI[STREAM] parameter - Align loader paths with XDG layout * chore: update gitignore, checksums, and docs - Exclude AI agent instruction files from git tracking (AGENTS.md, CLAUDE.md, GEMINI.md, .github/copilot-instructions.md) - Add .geminiignore - Regenerate lib/checksum.txt to reflect updated scripts - Document sync-init.sh usage in docs/README.md - Remove deprecated git.io short URLs from README * Checksum e470ca3 * ci: add org project tracker scripts (#158) * ci(check-linux): add init.zsh sync-drift detection step * fix(install.sh): verify zi.zsh exists after clone, not just directory * ci: add concurrency to checksum.yml; add smoke-test to win-install.yml Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * feat: add color.zsh from molovo/color (MIT) * feat: add revolver from molovo/revolver (MIT) * chore: rename repo references from zi-src to src Update all internal references to reflect the repository rename: - .trunk/trunk.yaml repo name - workflow repository guards (checksum, rclone-action, deploy-gh-pages) - win-install safe.directory path - docs/README.md VSCode and jsDelivr URLs * ci: add org project tracker scripts (label sync + workflow rollout) * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * fix: address review feedback for revolver process check and installer URLs Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com> * fix: validate revolver pid before liveness check Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com> --------- Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * chore(ci): remove emoji prefixes from workflow names * build(deps): bump the github-actions group across 1 directory with 5 updates (#173) Bumps the github-actions group with 5 updates in the / directory: | Package | From | To | | --- | --- | --- | | [actions/checkout](https://github.com/actions/checkout) | `6.0.2` | `7.0.1` | | [z-shell/.github/actions/commit](https://github.com/z-shell/.github) | `1.0.8` | `1.1.1` | | [peaceiris/actions-gh-pages](https://github.com/peaceiris/actions-gh-pages) | `4.0.0` | `4.1.0` | | [z-shell/.github/actions/rclone](https://github.com/z-shell/.github) | `1.0.8` | `1.1.1` | | [egor-tensin/setup-cygwin](https://github.com/egor-tensin/setup-cygwin) | `4.0.1` | `4.0.2` | Updates `actions/checkout` from 6.0.2 to 7.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@de0fac2...3d3c42e) Updates `z-shell/.github/actions/commit` from 1.0.8 to 1.1.1 - [Release notes](https://github.com/z-shell/.github/releases) - [Commits](z-shell/.github@91068ee...3222f53) Updates `peaceiris/actions-gh-pages` from 4.0.0 to 4.1.0 - [Release notes](https://github.com/peaceiris/actions-gh-pages/releases) - [Changelog](https://github.com/peaceiris/actions-gh-pages/blob/main/CHANGELOG.md) - [Commits](peaceiris/actions-gh-pages@4f9cc66...84c30a8) Updates `z-shell/.github/actions/rclone` from 1.0.8 to 1.1.1 - [Release notes](https://github.com/z-shell/.github/releases) - [Commits](z-shell/.github@91068ee...3222f53) Updates `egor-tensin/setup-cygwin` from 4.0.1 to 4.0.2 - [Release notes](https://github.com/egor-tensin/setup-cygwin/releases) - [Changelog](https://github.com/egor-tensin/setup-cygwin/blob/master/CHANGELOG.md) - [Commits](egor-tensin/setup-cygwin@d2c752b...fca9069) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: github-actions - dependency-name: z-shell/.github/actions/commit dependency-version: 1.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: peaceiris/actions-gh-pages dependency-version: 4.1.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: z-shell/.github/actions/rclone dependency-version: 1.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: egor-tensin/setup-cygwin dependency-version: 4.0.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: resolve merge conflicts with origin/main * fix: address code review issues from PR review - Fix mktemp -d portability: add template to all calls for BSD/macOS compat (tests/installers.sh, public/sh/install.sh, install_zpmod.sh, sync-init.sh, scripts/add-project-workflow.sh) - Fix add-project-workflow.sh: fail fast if mktemp fails - Fix _comps[zi] in public/zsh/init.zsh to register function name _zi instead of absolute file path (fixes completion loading) - Fix ANSI escape sequences in public/sh/sync-init.sh: use printf '%b' with real ESC prefix so color codes render correctly - Fix merged comment with duplicated # in public/zsh/snippets/revolver * fix: add completion dir to fpath in _zi_comps before registering _zi * revert: remove fpath manipulation from _zi_comps, defer to zi.zsh * chore: remove deprecated scripts/ directory * fix: update checksums to match current script files --------- Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Salvydas Lukosius <ss-o@users.noreply.github.com> Co-authored-by: digital-teams[bot] <actions@zshell.dev> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* chore: remove deprecated files and configs Remove obsolete cspell word lists, editorconfig, vscode settings, and retired workflows (rebase, stale, sync-labels, wrangler). Drop legacy installer scripts i.sh and install_alpha.sh. * ci: update workflows and trunk configuration - Bump actions/checkout to v6.0.2 and shellcheck to 2.0.0 - Add lib/zsh/** and tests/** to push/PR path triggers - Add permissions: contents: read and timeout-minutes: 10 - Use github.ref in concurrency group - Replace ~/.zi hard-coded paths with XDG_DATA_HOME equivalents - Add unit-fixture step calling tests/installers.sh - Remove wrangler deploy job; update trunk linter config * feat: add sync-init.sh and tests/installers.sh sync-init.sh verifies (and optionally replaces) the local lib/zsh/init.zsh against the canonical GitHub raw main copy. Supports --write, --local, --remote, --checksum-url and --no-checksum flags. tests/installers.sh provides a POSIX sh test suite covering: - script syntax validation (sh -n / zsh -n) - checksum integrity checks - loader install with XDG path and branch-override assertions - standalone zpmod delegation test - sync-init fixture round-trip test * refactor: migrate installer scripts to XDG paths and harden POSIX install.sh: - Add set -eu; use WORKDIR=$(mktemp -d) with safe trap cleanup - Replace ~/.zi with XDG_DATA_HOME-aware path - Fix sed command for ZI[STREAM] substitution - Correct cd fallback from return to exit 1 - Guard branch pull with BOPT; add exit 1 on failed clone - Fix .zshrc loader snippet variable quoting install_zpmod.sh: - Add set -eu, file header and WORKDIR trap - Add pure-POSIX _zi_ver_ge() replacing expr string comparison - Honour XDG_DATA_HOME for ZI_HOME default - Replace exit 255 with exit 1 throughout lib/zsh/init.zsh: - Update default branch to use ZI[STREAM] parameter - Align loader paths with XDG layout * chore: update gitignore, checksums, and docs - Exclude AI agent instruction files from git tracking (AGENTS.md, CLAUDE.md, GEMINI.md, .github/copilot-instructions.md) - Add .geminiignore - Regenerate lib/checksum.txt to reflect updated scripts - Document sync-init.sh usage in docs/README.md - Remove deprecated git.io short URLs from README * Checksum e470ca3 * ci: add org project tracker scripts (#158) * ci(check-linux): add init.zsh sync-drift detection step * fix(install.sh): verify zi.zsh exists after clone, not just directory * ci: add concurrency to checksum.yml; add smoke-test to win-install.yml Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * feat: add color.zsh from molovo/color (MIT) * feat: add revolver from molovo/revolver (MIT) * chore: rename repo references from zi-src to src Update all internal references to reflect the repository rename: - .trunk/trunk.yaml repo name - workflow repository guards (checksum, rclone-action, deploy-gh-pages) - win-install safe.directory path - docs/README.md VSCode and jsDelivr URLs * ci: add org project tracker scripts (label sync + workflow rollout) * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * fix: address review feedback for revolver process check and installer URLs Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com> * fix: validate revolver pid before liveness check Agent-Logs-Url: https://github.com/z-shell/src/sessions/3295a818-69b9-4b0d-99f8-bf27b90e3b7c Co-authored-by: ss-o <59910950+ss-o@users.noreply.github.com> --------- Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> * chore(ci): remove emoji prefixes from workflow names * build(deps): bump the github-actions group across 1 directory with 5 updates (#173) Bumps the github-actions group with 5 updates in the / directory: | Package | From | To | | --- | --- | --- | | [actions/checkout](https://github.com/actions/checkout) | `6.0.2` | `7.0.1` | | [z-shell/.github/actions/commit](https://github.com/z-shell/.github) | `1.0.8` | `1.1.1` | | [peaceiris/actions-gh-pages](https://github.com/peaceiris/actions-gh-pages) | `4.0.0` | `4.1.0` | | [z-shell/.github/actions/rclone](https://github.com/z-shell/.github) | `1.0.8` | `1.1.1` | | [egor-tensin/setup-cygwin](https://github.com/egor-tensin/setup-cygwin) | `4.0.1` | `4.0.2` | Updates `actions/checkout` from 6.0.2 to 7.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@de0fac2...3d3c42e) Updates `z-shell/.github/actions/commit` from 1.0.8 to 1.1.1 - [Release notes](https://github.com/z-shell/.github/releases) - [Commits](z-shell/.github@91068ee...3222f53) Updates `peaceiris/actions-gh-pages` from 4.0.0 to 4.1.0 - [Release notes](https://github.com/peaceiris/actions-gh-pages/releases) - [Changelog](https://github.com/peaceiris/actions-gh-pages/blob/main/CHANGELOG.md) - [Commits](peaceiris/actions-gh-pages@4f9cc66...84c30a8) Updates `z-shell/.github/actions/rclone` from 1.0.8 to 1.1.1 - [Release notes](https://github.com/z-shell/.github/releases) - [Commits](z-shell/.github@91068ee...3222f53) Updates `egor-tensin/setup-cygwin` from 4.0.1 to 4.0.2 - [Release notes](https://github.com/egor-tensin/setup-cygwin/releases) - [Changelog](https://github.com/egor-tensin/setup-cygwin/blob/master/CHANGELOG.md) - [Commits](egor-tensin/setup-cygwin@d2c752b...fca9069) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: github-actions - dependency-name: z-shell/.github/actions/commit dependency-version: 1.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: peaceiris/actions-gh-pages dependency-version: 4.1.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: z-shell/.github/actions/rclone dependency-version: 1.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: egor-tensin/setup-cygwin dependency-version: 4.0.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: resolve merge conflicts with origin/main * fix: address code review issues from PR review - Fix mktemp -d portability: add template to all calls for BSD/macOS compat (tests/installers.sh, public/sh/install.sh, install_zpmod.sh, sync-init.sh, scripts/add-project-workflow.sh) - Fix add-project-workflow.sh: fail fast if mktemp fails - Fix _comps[zi] in public/zsh/init.zsh to register function name _zi instead of absolute file path (fixes completion loading) - Fix ANSI escape sequences in public/sh/sync-init.sh: use printf '%b' with real ESC prefix so color codes render correctly - Fix merged comment with duplicated # in public/zsh/snippets/revolver * fix: add completion dir to fpath in _zi_comps before registering _zi * revert: remove fpath manipulation from _zi_comps, defer to zi.zsh * chore: remove deprecated scripts/ directory * fix: update checksums to match current script files --------- Signed-off-by: Sall <59910950+ss-o@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Salvydas Lukosius <ss-o@users.noreply.github.com> Co-authored-by: digital-teams[bot] <actions@zshell.dev> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> 47f0510
Adds maintenance scripts for the Z-Shell Tracker org project (#28):
Scripts
scripts/sync-labels.sh— applies the canonical 21-label set to all org repos viagh label create --forcescripts/add-project-workflow.sh— rolls out.github/workflows/project-tracker.ymlto every repo (opens PRs)scripts/repos.txt— auto-generated list of all 94 z-shell org reposRelated