Back to feed

sledtools/pika branch #79

delete-stale-todos

docs: delete stale todo plans

Target branch: master

Merge Commit: 53d554a3211bbc044caafd8fe09bbbb6f9fb7da0

branch: merged tutorial: ready ci: success
Open CI Details

Continuous Integration

CI: success

Compact status on the review page, with full logs on the CI page.

Open CI Details

Latest run #103 success

head a74f58660f9de6becaefcc2989f86c570c7e0f5d · queued 2026-03-26 00:26:06 · 0 lane(s)

queued 0s

No lanes were selected for this branch head.

Summary

This branch performs a documentation cleanup by deleting twelve stale planning and TODO documents from the todos/ directory. The removed files span several domains: an agents platform architecture document, four call-related implementation plans (audio, native audio, transport, video), a daemon product protocol closeout note, a faster CI strategy document, an iOS share extension plan, a Pika share plan, a staged CI plan, a shared test fixtures plan, and a living test plan. None of these deletions affect source code; they purely remove planning artifacts that had become outdated or were superseded by actual implementation work.

Tutorial Steps

Remove the agents platform architecture document

Intent: Delete the large north-star requirements and architecture document for the managed multi-agent hosting platform (`agents.pikachat.org`). This document covered tenancy models, control plane vocabulary, web dashboard requirements, authentication, update/drift control, and delivery sequencing — all of which have either been implemented, superseded, or are no longer relevant as active planning artifacts.

Affected files: todos/agents-platform.md

Evidence
@@ -1,675 +0,0 @@
-# Agents Platform
-
-This is a living product and architecture document for a managed multi-agent platform built on
-PikaChat and microVMs.

The agents-platform.md file was the largest document removed at 675 lines. It served as a directional companion for the managed VM hosting product, covering:

  • Core tenancy model (one customer per VM, multiple agents per VM)
  • Control plane vocabulary (CustomerVm, AgentInstance, AgentTemplate, etc.)
  • Web dashboard requirements and SSR technology direction
  • Authentication via Nostr login
  • Routing and origin separation for agent UIs
  • Update, drift control, and future eject mode
  • Delivery sequence and explicit non-goals

As a living document that was meant to evolve, its removal signals that the planning phase it represented has concluded or moved to other artifacts.

Remove the call audio codec plan

Intent: Delete the plan for replacing PCM passthrough with real Opus encode/decode in the Rust media core, along with adaptive jitter buffering and loss concealment.

Affected files: todos/call-audio-plan.md

Evidence
@@ -1,81 +0,0 @@
-# call-audio-plan.md
-
-Status: proposed (highest priority)
-
-This plan has no upstream dependency and is the prerequisite foundation for `call-transport-plan.md`, `call-video-plan.md`, and `call-native-audio-plan.md`.

This 81-line plan was marked as the highest priority call improvement and served as the prerequisite for the three other call plans. It covered:

  • Replacing codec_opus.rs PCM passthrough with real Opus bindings
  • Gap concealment and adaptive playout buffer logic
  • Tuning and regression test coverage

Key files referenced: crates/pika-media/src/codec_opus.rs, rust/src/core/call_runtime.rs, crates/pika-media/src/jitter.rs.

Remove the native mobile audio pipeline plan

Intent: Delete the detailed plan for migrating mobile call audio I/O from Rust `cpal` to native iOS (AVAudioEngine) and Android (AudioRecord/AudioTrack) pipelines while keeping transport, crypto, and call state in Rust.

Affected files: todos/call-native-audio-plan.md

Evidence
@@ -1,272 +0,0 @@
-# call-native-audio-plan.md
-
-Status: proposed (high bang-for-buck, RMP native-capability-bridge compliant)
-
-This plan depends on `call-audio-plan.md` for codec/jitter quality baselines

At 272 lines, this was the most detailed call plan. It specified:

  • A native capability bridge architecture keeping Rust as the media brain
  • iOS implementation using AVAudioEngine with voice processing
  • Android implementation using AudioRecord/AudioTrack with AEC/NS/AGC
  • FFI contract changes (AudioPlayoutReceiver, send_audio_capture_frame)
  • An eight-step incremental delivery sequence from contract freeze through manual QA
  • RMP (Rust Multiplatform) alignment guardrails

The plan explicitly referenced files like CallAudioSessionCoordinator.swift, AndroidAudioFocusManager.kt, and call_runtime.rs.

Remove the call transport resilience plan

Intent: Delete the plan for making active calls survive transient network disruptions through subscription readiness gating, mid-call reconnect/resubscribe loops, and transport health telemetry.

Affected files: todos/call-transport-plan.md

Evidence
@@ -1,85 +0,0 @@
-# call-transport-plan.md
-
-Status: proposed (second priority)
-
-This plan depends on `call-audio-plan.md`, should be completed before `call-video-plan.md`

This 85-line plan was the second priority in the call improvement sequence. It covered four workstreams:

  1. Startup reliability via MediaFrameSubscription::wait_ready()
  2. Reconnect and resubscribe state machine with bounded exponential backoff
  3. Delivery semantics and telemetry hardening
  4. Failure-mode test matrix for relay restart, delayed subscription, and disconnect recovery

Key files referenced: call_runtime.rs, crates/pika-media/src/network.rs, crates/pika-media/src/subscription.rs.

Remove the call video quality plan

Intent: Delete the plan for reducing frozen/black-video incidents through decoder recovery, keyframe discipline, and runtime video adaptation.

Affected files: todos/call-video-plan.md

Evidence
@@ -1,82 +0,0 @@
-# call-video-plan.md
-
-Status: proposed (third priority)
-
-This plan depends on `call-audio-plan.md` and `call-transport-plan.md`

This 82-line plan was the third priority in the call improvement chain. It specified:

  1. Decoder recovery path for iOS VideoDecoderRenderer instead of terminal failure on corrupt payloads
  2. Keyframe discipline with configurable cadence and request-keyframe signaling
  3. Video adaptation controls with quality levels driven by runtime stats
  4. Validation and regression coverage in deterministic and E2E tests

Key files referenced: VideoDecoderRenderer.swift, VideoCaptureManager.swift, call_runtime.rs.

Remove the daemon product protocol closeout note

Intent: Delete the closeout document for the daemon/app parity project, which documented what shipped, what was explicitly excluded, and why no further parity PR was recommended.

Affected files: todos/daemon-product-protocol.md

Evidence
@@ -1,148 +0,0 @@
-# Daemon Product Protocol
-
-This document is now a closeout note for the daemon/app parity project, not an active wishlist.

This 148-line document had already transitioned from an active plan to a closeout note. It cataloged:

  • The production surfaces that shipped (group management, messaging, media, calls)
  • Group observability via group_updated event family
  • Group profile contract details
  • Explicit non-goals (protocol JSON shape, event fanout, reconnect policy)
  • Justification for closing the project

Since the document itself declared the project closed, removing it is a natural cleanup step.

Remove the faster CI strategy document

Intent: Delete the strategy document outlining ideas for dramatically improving CI speed through finer-grained parallelism, persistent runner pools, and longest-first scheduling.

Affected files: todos/faster-ci.md

Evidence
@@ -1,108 +0,0 @@
-# Faster CI
-
-Ideas for making pikaci runs significantly faster, mostly through finer-grained parallelism.

This 108-line document proposed seven strategies for CI improvement:

  1. Instrument per-binary timing
  2. Consolidate runner flakes into one runner image per target
  3. One binary per VM for pure tests
  4. Deferred pikahut-dependent test parallelization
  5. Persistent runner pool (pre-provisioned VMs on pika-build)
  6. Longest-first scheduling with bin-packing
  7. Identifying and splitting slow binaries

It included concrete projections (e.g., 71s sequential → 25s wall clock with 4 runners) and discussed Incus considerations and Apple path options.

Remove the iOS share extension plan

Intent: Delete the planning document for the iOS share extension feature.

Affected files: todos/ios-share-extension-plan.md

Evidence
@@ -1,81 +0,0 @@

This file contained the plan for implementing an iOS share extension for Pika. The full diff content was truncated in the input, but the file was 81 lines and has been removed as a stale planning artifact.

Remove the Pika share plan

Intent: Delete the planning document for the Pika share feature.

Affected files: todos/pika-share-plan.md

Evidence
@@ -1,81 +0,0 @@

This planning document covered the Pika share feature design. Like the iOS share extension plan, its full content was truncated in the diff input. It has been removed as part of the stale TODO cleanup.

Remove the staged CI plan

Intent: Delete the planning document for pikaci staged CI improvements.

Affected files: todos/pikaci-staged-ci-plan.md

Evidence
@@ -1,81 +0,0 @@

This document outlined a staged approach to CI pipeline improvements for pikaci. It has been removed alongside the related faster-ci.md document, suggesting that CI strategy has either been implemented or has moved to different planning artifacts.

Remove the shared test fixtures plan

Intent: Delete the planning document for pikahut shared test fixtures.

Affected files: todos/pikahut-shared-fixtures.md

Evidence
@@ -1,81 +0,0 @@

This document covered plans for shared test fixtures in the pikahut testing infrastructure. Its removal indicates the testing strategy it described has either been realized or is no longer the intended approach.

Remove the living test plan

Intent: Delete the living test plan document that tracked the evolving testing strategy.

Affected files: todos/tests-living-plan.md

Evidence
@@ -1,81 +0,0 @@

This was a living document tracking the testing strategy across the project. As with the other removed files, it has been deleted as a stale planning artifact. Living documents that are no longer actively maintained become misleading over time, making this cleanup valuable for reducing documentation drift.

Diff