Skip to content

chore: delete dead code (PR 1/3) — -29,458 lines #167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
EtienneLescot merged 6 commits into from
Jul 26, 2026
Merged

chore: delete dead code (PR 1/3) — -29,458 lines #167

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
cd6bdcc
refactor(exporter): delete the web MP4 pipeline the native path replaced
EtienneLescot Jul 26, 2026
366f097
chore: drop the two superseded POCs and their spike residue
EtienneLescot Jul 26, 2026
cf7e667
docs: drop consumed one-shot plans and a stale tracked worktree handoff
EtienneLescot Jul 26, 2026
e909e0e
chore(deps): drop seven unused dependencies and a ghost CI cache
EtienneLescot Jul 26, 2026
f0d413b
fix(export): remove the hardcoded fixture path from the shipped addon
EtienneLescot Jul 26, 2026
8c9b832
docs: repoint the export and preview docs at what survives the deletion
EtienneLescot Jul 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions .claude/launch.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,6 @@
"runtimeExecutable": "node",
"runtimeArgs": ["node_modules/vite/bin/vite.js", "--config", "vite.v4preview.config.ts"],
"port": 5207
},
{
"name": "poc",
"runtimeExecutable": "npx",
"runtimeArgs": ["vite", "--config", "poc/vite.config.js"],
"port": 5210
}
]
}
24 changes: 0 additions & 24 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,6 @@ jobs:
- name: Setup Node.js
uses: ./.github/actions/setup

- name: Cache caption assets
uses: actions/cache@v4
with:
path: caption-assets
key: caption-assets-${{ runner.os }}-${{ hashFiles('scripts/fetch-caption-model.mjs') }}

# ponytail: STT is handled by the bundled whisper-stt-server (whisper.cpp
# with native DTW token timestamps). No separate VAD model is fetched
# here. See technical-documentation/architecture/transcription-and-captions.md for the full architecture.
Expand All @@ -69,12 +63,6 @@ jobs:
- name: Setup Node.js
uses: ./.github/actions/setup

- name: Cache caption assets
uses: actions/cache@v4
with:
path: caption-assets
key: caption-assets-${{ runner.os }}-${{ hashFiles('scripts/fetch-caption-model.mjs') }}

# ponytail: STT is handled by the bundled whisper-stt-server (whisper.cpp
# with native DTW token timestamps). No separate VAD model is fetched
# here. See technical-documentation/architecture/transcription-and-captions.md for the full architecture.
Expand Down Expand Up @@ -114,12 +102,6 @@ jobs:
env:
npm_config_build_from_source: "false"

- name: Cache caption assets
uses: actions/cache@v4
with:
path: caption-assets
key: caption-assets-${{ runner.os }}-${{ hashFiles('scripts/fetch-caption-model.mjs') }}

# ponytail: STT is handled by the bundled whisper-stt-server (see the
# build-windows comment); no VAD model is fetched here.
- name: Resolve macOS signing
Expand Down Expand Up @@ -279,12 +261,6 @@ jobs:
- name: Install pacman build dependencies
run: sudo apt-get update && sudo apt-get install -y libarchive-tools

- name: Cache caption assets
uses: actions/cache@v4
with:
path: caption-assets
key: caption-assets-${{ runner.os }}-${{ hashFiles('scripts/fetch-caption-model.mjs') }}

# ponytail: no VAD fetch here either (see build-windows comment).
- name: Build Linux app
run: npm run build:linux -- --publish never
Expand Down
Loading
Loading