Skip to content

Remove stale onboarding code after flow simplification#1429

Merged
tyler-dane merged 5 commits intomainfrom
copilot/remove-stale-onboarding-code
Feb 3, 2026
Merged

Remove stale onboarding code after flow simplification#1429
tyler-dane merged 5 commits intomainfrom
copilot/remove-stale-onboarding-code

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

The onboarding was recently simplified from a multi-step interactive flow to a command palette guide overlay. This left ~81 files of unused code (steps, layouts, sandboxes, tutorials).

Changes

Removed (79 files, 12K LOC)

  • steps/ directory - welcome screens, sign-in flow, task/event sandboxes, reminder setup, outro screens
  • Component layer - Onboarding.tsx container, OnboardingStep, footers, forms, overlays, layouts, styled-components
  • Hook layer - useOnboardingShortcuts, useOnboardingOverlay, useStoredTasks, useAuthPrompt
  • Types - onboarding-notice.types.ts and barrel files

Modified

  • useGoogleAuth.ts - removed unused OnboardingStepProps parameter

Retained (15 files)

  • OnboardingGuide component and supporting UI components
  • Step detection, state management, and progress tracking hooks
  • Storage utilities and step configuration constants

Example

Before (unused):

export function useGoogleAuth(props?: OnboardingStepProps) {
  // ...
  props?.onNext?.(); // Called old multi-step flow
}

After:

export function useGoogleAuth() {
  // Simplified - no step props needed for guide overlay
}
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove stale onboarding code</issue_title>
<issue_description>We recently simplified onboarding, meaning there are now a lot of unused component code in src/views/Onboarding and elsewhere. Remove all the dead code related to onboarding</issue_description>

<agent_instructions>Be liberal in removing code if it doesn't have a use</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove stale onboarding code from the project Remove stale onboarding code after flow simplification Feb 2, 2026
Copilot AI requested a review from tyler-dane February 2, 2026 01:17
@tyler-dane tyler-dane marked this pull request as ready for review February 3, 2026 03:23
Copilot AI review requested due to automatic review settings February 3, 2026 03:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes approximately 81 files (~12K LOC) of unused onboarding code following the simplification of the onboarding flow from an interactive multi-step process to a command palette guide overlay.

Changes:

  • Removes 79 unused files including steps, layouts, sandboxes, tutorials, hooks, types, and tests
  • Modifies useGoogleAuth.ts to remove the unused OnboardingStepProps parameter
  • Retains 15 files that support the new guide overlay system

Reviewed changes

Copilot reviewed 80 out of 80 changed files in this pull request and generated no comments.

File Description
useGoogleAuth.ts Removes unused OnboardingStepProps parameter and onNext callback, simplifying the function signature
All other files Deletion of unused onboarding components, hooks, types, tests, and utilities from the old multi-step flow

@tyler-dane tyler-dane merged commit 00eb964 into main Feb 3, 2026
11 checks passed
@tyler-dane tyler-dane deleted the copilot/remove-stale-onboarding-code branch February 3, 2026 03:25
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.

Remove stale onboarding code

2 participants