Skip to content

chore: remove companion app (moved to calcom/companion)#27957

Merged
keithwillcode merged 1 commit intomainfrom
devin/1771098332-remove-companion
Feb 16, 2026
Merged

chore: remove companion app (moved to calcom/companion)#27957
keithwillcode merged 1 commit intomainfrom
devin/1771098332-remove-companion

Conversation

@volnei
Copy link
Contributor

@volnei volnei commented Feb 14, 2026

What does this PR do?

Removes the companion/ directory and all associated CI configuration from this monorepo. The companion app (iOS, Android, browser extension) has been migrated to its own repository at calcom/companion with full commit history preserved.

Changes

  • Deleted companion/ — all source code, configs, and assets
  • Deleted companion CI workflowscompanion-build.yml, companion-lint.yml, companion-typecheck.yml
  • Deleted .github/ISSUE_TEMPLATE/companion.md
  • Deleted .easignore — EAS Build ignore file (was solely for companion mobile builds)
  • Updated .github/workflows/pr.yml — removed has-companion output, path filter, three companion CI jobs (build-companion, typecheck-companion, lint-companion), and their references in the required job's needs list and failure conditions
  • Updated packages/lib/timeZones.ts — comment now points to calcom/companion repo instead of local companion/ path

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A — deletion only, no docs impact.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A — deletion only; CI passing confirms no regressions.

How should this be tested?

  1. Verify CI passes on this PR — the companion jobs should no longer appear in new workflow runs
  2. Confirm no remaining references to companion/ in the repo outside of unrelated contexts (e.g., the Alby app-store description mentioning "companion" is unrelated)

Human Review Checklist

  • pr.yml changes (most important) — Verify the required job's needs list and failure conditions are still structurally correct after removing the companion block. The conditional logic around has-agents-changes at the end of the failure check should still be properly connected.
  • .easignore deletion — Confirm nothing else in the monorepo depends on this file
  • No orphaned references — Verify no other CI workflows or configs still reference companion/

Note on expected CI behavior: The required check will fail on this PR because pull_request_target runs the workflow from main, which still defines the companion jobs. Since companion code is deleted in the PR, those jobs fail. This is a chicken-and-egg situation that resolves once merged. All code-specific checks (lint, type-check, unit tests) pass.

Link to Devin run: https://app.devin.ai/sessions/c426fbde722b4a668333a803306fd36e
Requested by: @volnei

Co-Authored-By: Volnei Munhoz <volnei.munhoz@gmail.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 301 files

Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.

@volnei volnei marked this pull request as ready for review February 14, 2026 20:03
@volnei volnei requested a review from a team as a code owner February 14, 2026 20:03
@volnei volnei enabled auto-merge (squash) February 14, 2026 20:03
@graphite-app graphite-app bot added foundation core area: core, team members only labels Feb 14, 2026
@github-actions
Copy link
Contributor

E2E results are ready!

@keithwillcode
Copy link
Contributor

Have to force merge because GitHub doesn’t use the updated GitHub actions here. It uses main instead so the checks fail

@keithwillcode keithwillcode merged commit f8a9341 into main Feb 16, 2026
83 of 94 checks passed
@keithwillcode keithwillcode deleted the devin/1771098332-remove-companion branch February 16, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only foundation ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants