mirror of
https://github.com/mattermost/mattermost-mobile.git
synced 2026-11-04 09:58:57 +00:00
0f9beb0f95
* ci(e2e): per-job TSIO report groups and commit statuses E2E jobs previously shared a single `e2e/mobile` commit status backed by one TSIO report group, so concurrent platform legs raced to finalize the group and clobbered each other's results. Give each Detox/Maestro job its own report group (`mobile-<flow>-<job>`, total_reports_expected=1) and its own commit status context (`detox-ios`, `detox-android`, `detox-ipad`, `maestro-ios`, `maestro-android`); CMT reports per shard plus an umbrella context. Channel notifications move to a single rollup step so N platform legs no longer spam the webhook, and the five-server topology is dispatched with explicit per-platform inputs that still resolve to the existing SITE_*_URL contract consumed by the test config. Also adds the webhook sidecar launcher, Android emulator CI prep, and TSIO screenshot path linking that keeps folder basenames matching Jest full names. The new detox type-check job surfaced pre-existing broken import paths in maestro/fixtures/calls_seed.ts, which pointed at detox/detox/e2e; corrected so the guard passes on its own first run. Note: the CMT matrix now reads per-platform server URLs (`ios_site_1_url`, `android_site_1_url`, `site_3_url`) instead of a single `url`, which needs the matching Matterwick dispatch change to land alongside this. Co-authored-by: Cursor <cursoragent@cursor.com> * ci(e2e): aggregate CMT channel rollup from TSIO and wire notify secrets The CMT umbrella posted a placeholder summary built from CI job outcomes (passed:0, failed:0|1, reports:[]), so the release channel message carried no per-leg detail. It now runs the same tsio-channel-notify-rollup.js path as PR/main: cmt-identity emits a per-shard TSIO config map and the rollup polls every shard group, so counts and per-leg rows come from TSIO. The rollup job also needs id-token: write to mint its OIDC token. Webhook secrets used by the per-job TSIO finalize steps were referenced but never declared in the templates' workflow_call.secrets, so GitHub resolved them to empty strings and channel notify silently no-opped. Declare them in the iOS/Android/Maestro templates and forward them from e2e-detox.yml and the CMT callers. The Maestro PR path worked only because its callers use secrets: inherit. Also from review: - split-tests.js emitted two matrix jobs at parallelism=1 when mm_blocks specs were present; merge into one shard instead of exceeding the requested count. - Move the Android ARTIFACT_RUN_ID emptiness check ahead of download-artifact, which consumes it and otherwise fails with an opaque API error. - resolve-tsio-job-config.sh now reuses webhookBucketForReportName instead of re-deriving the bucket prefix with a looser startsWith check. Co-authored-by: Cursor <cursoragent@cursor.com> * ci(e2e): add local preflight for CI PRs The webhook-secret and OIDC-permission gaps in this branch were invisible locally and in a green CI run — the affected steps exit 0 and just skip the notify or commit status. Add a script engineers can run before opening a PR that touches .github/ or detox/utils: cd detox && npm run preflight:ci # :ci-fast skips lint + tsc It runs actionlint (which flags secrets not declared in a callee's workflow_call.secrets), scans for exit-code capture broken by a trailing backslash, asserts every job running a TSIO OIDC script declares id-token: write, then runs the detox/utils tests and npm run check. actionlint and the CodeRabbit CLI are optional and skipped with an install hint when absent. Co-authored-by: Cursor <cursoragent@cursor.com> * ci(e2e): use mktemp for preflight logs Fixed /tmp log paths can be pre-created as symlinks, so the preflight script would truncate whatever they point at (CWE-377). Create both logs with mktemp and drop them on exit. Co-authored-by: Cursor <cursoragent@cursor.com> * ci(e2e): address review feedback on TSIO statuses and sidecar A cancelled E2E run reached no conclusion, so posting `success` on its commit statuses let a PR merge on a check that never ran. Cancelled runs now post `pending`; `success` is reserved for the one deliberate bypass, the E2E/Override label. Commit-status contexts move into the `e2e-test/*` namespace to match the mattermost monorepo, with the prefix owned by build-tsio-job-config.js and mirrored in the cancel action's context list. Legacy unprefixed and `e2e/*` contexts are dropped rather than carried forward. The `preflight:ci` script and its OIDC-permission checker duplicated what actionlint, the unit tests and `npm run check` already do, so they are removed and detox/CLAUDE.md documents the underlying failure modes and the commands to catch them instead. cloudflared and the TSIO CLI are pinned to known-good releases rather than `latest`, and detox/README.md now documents the webhook sidecar: when a tunnel is needed, the WEBHOOK_PUBLIC_BASE_URL / CLOUDFLARED_TUNNEL_TOKEN precedence, and why quick tunnels report callbacks as unreachable. Co-authored-by: Cursor <cursoragent@cursor.com> * ci(e2e): detect upstream cancellation and surface status reset failures generate-report runs with always(), so job.status stays "success" when an upstream job is cancelled and TSIO was published as a real failure. Read the generate-specs/e2e-* needs results so a cancelled run is reported as such. In cancel-e2e-runs, only reset the E2E commit statuses for runs that were actually stopped — a run we failed to cancel keeps publishing its own results. Aggregate createCommitStatus failures across every context and fail the action afterwards instead of silently warning, so a stale check cannot go unnoticed. --------- Co-authored-by: Cursor <cursoragent@cursor.com>
147 lines
2.2 KiB
Plaintext
147 lines
2.2 KiB
Plaintext
assets/override
|
|
dist
|
|
build-ios
|
|
*.zip
|
|
server.PID
|
|
mattermost.keystore
|
|
tmp/
|
|
.env
|
|
env.d.ts
|
|
*.apk
|
|
*.aab
|
|
*.ipa
|
|
.tsbuildinfo.precommit
|
|
|
|
*/**/compass-icons.ttf
|
|
|
|
# OSX
|
|
#
|
|
.DS_Store
|
|
|
|
# Xcode
|
|
#
|
|
ios/build/*
|
|
*.pbxuser
|
|
!default.pbxuser
|
|
*.mode1v3
|
|
!default.mode1v3
|
|
*.mode2v3
|
|
!default.mode2v3
|
|
*.perspectivev3
|
|
!default.perspectivev3
|
|
xcuserdata
|
|
*.xccheckout
|
|
*.moved-aside
|
|
DerivedData
|
|
*.hmap
|
|
*.xcuserstate
|
|
project.xcworkspace
|
|
ios/Pods
|
|
.podinstall
|
|
ios/.xcode.env.local
|
|
|
|
# Android/IntelliJ
|
|
#
|
|
.idea
|
|
.gradle
|
|
local.properties
|
|
*.iml
|
|
*.hprof
|
|
.cxx/
|
|
*.keystore
|
|
!debug.keystore
|
|
android/app/bin
|
|
android/app/build
|
|
android/build
|
|
.settings
|
|
.project
|
|
.classpath
|
|
.kotlin
|
|
|
|
# node.js
|
|
#
|
|
node_modules/
|
|
npm-debug.log
|
|
.npminstall
|
|
yarn-error.log
|
|
.yarninstall
|
|
|
|
# Vim
|
|
[._]*.s[a-w][a-z]
|
|
[._]s[a-w][a-z]
|
|
*.un~
|
|
Session.vim
|
|
.netrwhist
|
|
*~
|
|
tags
|
|
|
|
# fastlane
|
|
#
|
|
# It is recommended to not store the screenshots in the git repo. Instead, use fastlane to re-generate the
|
|
# screenshots whenever they are needed.
|
|
# For more information about the recommended setup visit:
|
|
# https://docs.fastlane.tools/best-practices/source-control/
|
|
|
|
**/fastlane/report.xml
|
|
**/fastlane/Preview.html
|
|
**/fastlane/screenshots
|
|
**/fastlane/test_output
|
|
**/fastlane/.env
|
|
|
|
# Sentry
|
|
android/sentry.properties
|
|
ios/sentry.properties
|
|
|
|
# Testing
|
|
.nyc_output
|
|
coverage
|
|
.tmp
|
|
|
|
# E2E testing
|
|
mattermost-license.txt
|
|
*.mattermost-license
|
|
detox/artifacts
|
|
detox/detox_pixel_*
|
|
detox/e2e/support/fixtures/*.tar.gz
|
|
|
|
# Bundle artifact
|
|
*.jsbundle
|
|
.bundle
|
|
|
|
#editor-settings
|
|
.vscode
|
|
.scannerwork
|
|
launch.json
|
|
|
|
# Notice.txt generation
|
|
!build/notice-file
|
|
|
|
# Temporary files created by Metro to check the health of the file watcher
|
|
.metro-health-check*
|
|
|
|
libraries/**/**/build
|
|
libraries/**/**/.build
|
|
|
|
# Android sounds
|
|
android/app/src/main/res/raw/*
|
|
.aider*
|
|
|
|
# Intune submodule (private repository - internal builds only)
|
|
# Track submodule reference but ignore content
|
|
libraries/@mattermost/intune/*
|
|
!libraries/@mattermost/intune/.gitkeep
|
|
|
|
# Prevent committing Intune module in node_modules
|
|
node_modules/@mattermost/intune
|
|
# Claude Code
|
|
.claude/settings.local.json
|
|
.codex/
|
|
.cursor/
|
|
.planning/
|
|
|
|
# Local Maestro test env (generated by fixtures/seed.ts)
|
|
detox/maestro/.maestro-test-env*.sh
|
|
detox/maestro/.maestro-test-env*.bak
|
|
|
|
.e2e-debug/
|