Skip to content

Refactor autify-cli into a subdirectory so it works with workspaces#697

Merged
austin-autify merged 6 commits into
mainfrom
austin-autify/fix-changeset-action
May 13, 2026
Merged

Refactor autify-cli into a subdirectory so it works with workspaces#697
austin-autify merged 6 commits into
mainfrom
austin-autify/fix-changeset-action

Conversation

@austin-autify
Copy link
Copy Markdown
Contributor

@austin-autify austin-autify commented May 12, 2026

Include autify-cli in the workspaces to resolve issues with the changesets action where it can't find it: https://github.com/autifyhq/autify-cli/actions/runs/25713140101/job/75497411933

This change became a larger refactor because autify-cli wasn't properly included as a package in the existing workspace, and it doesn't work right when we include the root in the workspace.

This PR is probably best reviewed by ignoring the commit that moves all the files: https://github.com/autifyhq/autify-cli/pull/697/changes/e5c540b37f2e00ff14cc4a2cfd990c5d603e9281..76d99f94fb8a616866ec95e8814e167cbef6188c

@austin-autify austin-autify requested review from tai2 and xiankai May 12, 2026 05:26
Comment thread package-lock.json
upgrade lockfile to v3 to remove bloat
@austin-autify austin-autify force-pushed the austin-autify/fix-changeset-action branch from 2af2742 to 8da76ac Compare May 12, 2026 05:54
@austin-autify austin-autify marked this pull request as draft May 12, 2026 07:16
@austin-autify austin-autify force-pushed the austin-autify/fix-changeset-action branch 2 times, most recently from 9de2464 to dd77ec6 Compare May 12, 2026 07:34
@austin-autify austin-autify force-pushed the austin-autify/fix-changeset-action branch from dd77ec6 to 83d13de Compare May 12, 2026 07:44
@austin-autify austin-autify changed the title Fix changesets action by adding autify-cli to workspaces Refactor autify-cli into a subdirectory so it works with workspaces May 12, 2026
@austin-autify austin-autify marked this pull request as ready for review May 12, 2026 08:30
Copy link
Copy Markdown
Contributor

@xiankai xiankai left a comment

Choose a reason for hiding this comment

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

nice, the windows build is fine now

SHA: ${{ github.event.client_payload.sha }}
run: |
FILE="src/autify/mobile/mobilelink/mobilelink.json"
FILE="autify-cli/src/autify/mobile/mobilelink/mobilelink.json"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minor nit but seems possible to use working-directory here

publish: bash scripts/changesets-detect-publish.sh
# version and publish must be single commands (action splits on whitespace).
version: bash autify-cli/scripts/changesets-version.sh
publish: bash autify-cli/scripts/changesets-detect-publish.sh
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I also wonder if we can use working-directory here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pretty sure this one needs to run at the root where the .changesets dir is/workspace root

@austin-autify austin-autify merged commit e250ddb into main May 13, 2026
90 of 94 checks passed
@austin-autify austin-autify deleted the austin-autify/fix-changeset-action branch May 13, 2026 00:34
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.

2 participants