Improve CI master workflow

.github/workflows/build-and-test-pr.yml

@@ -14,7 +14,7 @@ permissions:
 jobs:
  determine-affected:
  name: "Turbo Affected"
- if: ${{github.event.pull_request.head.repo.full_name == github.repository }}
+ if: ${{!github.event.pull_request.head.repo.fork }}
  uses: LedgerHQ/ledger-live/.github/workflows/turbo-affected-reusable.yml@develop
  with:
  head_branch: ${{ github.event.pull_request.head.ref || github.event.merge_group.head_ref }}
@@ -24,65 +24,65 @@ jobs:
  build-desktop:
  name: "Build Desktop"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-desktop') && github.event.pull_request.head.repo.full_name == github.repository }}
- uses: LedgerHQ/ledger-live/.github/workflows/build-desktop-reusable.yml@develop
+ if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-desktop') && !github.event.pull_request.head.repo.fork }}
+ uses: LedgerHQ/ledger-live/.github/workflows/build-desktop-reusable.yml@fix/ci-master-workflow-inputs
  secrets: inherit
 
  test-desktop:
  name: "Test Desktop"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-desktop') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-desktop') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-desktop-reusable.yml@develop
  secrets: inherit
 
  # LLM
  build-mobile:
  name: "Build Mobile"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && github.event.pull_request.head.repo.full_name == github.repository}}
- uses: LedgerHQ/ledger-live/.github/workflows/build-mobile-reusable.yml@develop
+ if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && !github.event.pull_request.head.repo.fork}}
+ uses: LedgerHQ/ledger-live/.github/workflows/build-mobile-reusable.yml@fix/ci-master-workflow-inputs
  secrets: inherit
 
  test-mobile:
  name: "Test Mobile"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-mobile-reusable.yml@develop
  secrets: inherit
 
  test-mobile-e2e:
  name: "Test Mobile E2E"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'ledger-live-mobile') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-mobile-e2e-reusable.yml@develop
  secrets: inherit
 
  # Tests
  test-libraries:
  name: "Test Libraries"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'libs') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'libs') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-libs-reusable.yml@develop
  secrets: inherit
 
  test-design-system:
  name: "Test Design System"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'libs/ui') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'libs/ui') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-design-system-reusable.yml@develop
  secrets: inherit
 
  test-web-tools:
  name: "Test Web Tools"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'apps/web-tools') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'apps/web-tools') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-web-tools-reusable.yml@develop
  secrets: inherit
 
  test-cli:
  name: "Test CLI"
  needs: determine-affected
- if: ${{contains(needs.determine-affected.outputs.paths, 'apps/cli') && github.event.pull_request.head.repo.full_name == github.repository}}
+ if: ${{contains(needs.determine-affected.outputs.paths, 'apps/cli') && !github.event.pull_request.head.repo.fork}}
  uses: LedgerHQ/ledger-live/.github/workflows/test-cli-reusable.yml@develop
  secrets: inherit
 

.github/workflows/build-desktop-reusable.yml

@@ -19,10 +19,6 @@ on:
  description: The base branch to merge the head into when checking out the code
  type: string
  required: false
- prNumber:
- description: PR number
- type: string
- required: false
 
 permissions:
  id-token: write
@@ -155,11 +151,11 @@ jobs:
  return JSON.stringify(obj);
 
  - uses: LedgerHQ/ledger-live/tools/actions/build-checks@develop
- if: ${{ !cancelled() && inputs.prNumber != '' }}
+ if: ${{ !cancelled() && github.event.number != '' }}
  with:
  token: ${{ steps.generate-token.outputs.token }}
  baseBranch: ${{ inputs.base_ref || 'develop' }}
- prNumber: ${{ inputs.prNumber }}
+ prNumber: ${{ github.event.number}}
  mode: desktop
 
  - uses: LedgerHQ/ledger-live/tools/actions/desktop-report-build@develop

.github/workflows/build-mobile-reusable.yml

@@ -19,10 +19,6 @@ on:
  description: The base branch to merge the head into when checking out the code
  type: string
  required: false
- prNumber:
- description: PR number
- type: string
- required: false
 
 permissions:
  id-token: write
@@ -111,8 +107,8 @@ jobs:
  with:
  max_attempts: 2
  timeout_minutes: 15
- command: pnpm i --filter="live-mobile..." --filter="ledger-live" --no-frozen-lockfile --unsafe-perm 
- new_command_on_retry: rm -rf ~/.cocoapods/ && pnpm clean && pnpm i --filter="live-mobile..." --filter="ledger-live" --no-frozen-lockfile --unsafe-perm 
+ command: pnpm i --filter="live-mobile..." --filter="ledger-live" --no-frozen-lockfile --unsafe-perm
+ new_command_on_retry: rm -rf ~/.cocoapods/ && pnpm clean && pnpm i --filter="live-mobile..." --filter="ledger-live" --no-frozen-lockfile --unsafe-perm
  - name: bundle ios and android js
  run: |
  pnpm build:llm:deps --api="http://127.0.0.1:${{ steps.caches.outputs.port }}" --token="${{ secrets.TURBOREPO_SERVER_TOKEN }}" --team="foo"
@@ -144,11 +140,11 @@ jobs:
  with:
  name: mobile.metafile.json
  - uses: LedgerHQ/ledger-live/tools/actions/build-checks@develop
- if: inputs.prNumber != ''
+ if: github.event.number != ''
  with:
  token: ${{ steps.generate-token.outputs.token }}
  baseBranch: ${{ inputs.base_ref || 'develop' }}
- prNumber: ${{ inputs.prNumber }}
+ prNumber: ${{ github.event.number}}
  mode: mobile
  - uses: actions/github-script@v7
  name: get statuses of runs

libs/ledger-key-ring-protocol/src/api.ts

@@ -65,6 +65,7 @@
 };
 
 const getApi = (apiBaseURL: string) => {
+ console.log("hello");
  async function getAuthenticationChallenge(): Promise<{ json: Challenge; tlv: string }> {
  const { data } = await network<{ json: Challenge; tlv: string }>({
  url: `${apiBaseURL}/v1/challenge`,