Skip to content

ci: add developer id release workflow - #2

Merged
ai-eks merged 2 commits into
mainfrom
aix-add-source-release-workflow
Jul 19, 2026
Merged

ci: add developer id release workflow#2
ai-eks merged 2 commits into
mainfrom
aix-add-source-release-workflow

Conversation

@ai-eks

@ai-eks ai-eks commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add the Developer ID DMG release workflow directly to the MousePortal source repository.
  • Add the Developer ID entitlements file required by the package script.
  • Add the Developer ID DMG packaging script with source-root paths adjusted for this repository.

Why

The previous release workflow lived in MousePortal-dev and checked out MousePortal as a submodule. That fails when the source repository is private unless a SUBMODULE_TOKEN is configured. Keeping the release workflow in the source repository avoids the submodule checkout step entirely and lets tags in ai-eks/MousePortal build releases directly.

Validation

  • bash -n tools/package-developer-id.sh
  • git diff --check origin/main...HEAD
  • swift test

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 600060b2a9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/release-developer-id.yml

ai-eks commented Jul 19, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@ai-eks
ai-eks merged commit 7ffa58e into main Jul 19, 2026
1 check passed

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cb3ab7609d

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

xcrun stapler validate "$DMG_PATH"
fi

shasum -a 256 "$DMG_PATH" > "$DMG_PATH.sha256"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3 Badge Use portable asset names in checksum files

Because the default DIST_DIR is absolute, this writes the CI path (for example /.../dist/developer-id/MousePortal-1.0.0.dmg) into the released .sha256 file. I checked shasum --help: -c verifies a former output and uses the printed FILE name, so users who download the .dmg and .sha256 into any other directory will have shasum -c fail even though the asset is present. Generate the checksum from inside DIST_DIR or rewrite the filename to $(basename "$DMG_PATH").

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Followed up in #3 with a surgical fix.

The checksum is now generated from inside DIST_DIR, so the .sha256 file records MousePortal-<version>.dmg instead of the absolute CI path. Validation covered bash -n tools/package-developer-id.sh, git diff --check, and a temporary shasum -c smoke test.

@ai-eks
ai-eks deleted the aix-add-source-release-workflow branch July 19, 2026 12:29
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.

1 participant