jpmschweitzer/clide
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
Files
5301c4753cf1336d38fc499e172daef1fa120eb5
clide/decisions/rejected.md
T
jpmschweitzerandClaude Opus 4.6 2235cc4c19 drop leading zeros from all D/Q/R/T IDs 
pql 1.2.0 changed its record format to drop leading zeros. All
262 references across 18 files renumbered (D-001→D-1, T-043→T-43,
etc.). pql-plan.json export updated as the git source of truth
for the planning database. DB rebuilt via pql plan import.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-23 12:16:52 +02:00

5.8 KiB
Raw Blame History

Rejected Alternatives

Alternatives considered and rejected, with rationale preserved for future reference.

R-2: Go sidecar

  • Rejected: 2026-04-20 (was ADR 0002; superseded by D-5)
  • Reason: The ADR picked Go on two premises — (a) the heavy work belongs in a language separate from the UI layer, and (b) pql is Go so muscle memory transfers. Both broke on reassessment. The sidecar stripped of PTY is I/O-bound glue that dart:io covers cleanly (unix sockets, JSON-lines framing, process tables, shell-outs). The real axis was separate process vs shared language, not Go vs Rust, and separate-process is what matters (session persistence needs the daemon to outlive the app), not language. PTY is the one place Dart is genuinely weak — Dart's multi-threaded VM can't safely fork() — and that single constraint forces a native helper regardless, independent of whether the rest of the core is Dart. Once a small native helper is accepted, the question "does everything else need to be in that same native language" answers itself: no. Go sidecar directory dissolved; ptyc (C, PTY-only, pql-peer) is the surviving native supporter tool.
  • Cross-reference: D-5

R-3: MaterialApp root

  • Rejected: 2026-04-21
  • Reason: Dragged in Material theming, default icons, and platform chrome that fought the custom three-tier theme pipeline (D-9). Every bundled theme had to override Material defaults to look like clide; the overrides were visible in widget tests as "why is this ElevatedButton colored this way."
  • Cross-reference: D-7

R-4: Flutter intl + ARB codegen for i18n

  • Rejected: 2026-04-21
  • Reason: ARB codegen is inflexible for plugin-contributed catalogs — every catalogue needs a codegen pass, every extension ships with pre-generated Dart, and runtime merging is fighting the tool. The fframe text-driven pattern reads JSON at runtime with no codegen, which fits extension-shipped catalogs cleanly.
  • Cross-reference: D-21

R-5: Patrol test runner

  • Rejected: 2026-04-21
  • Reason: Adds a dependency (violates D-31) for a capability we get from Playwright + Flutter's own semantics tree. Patrol's value proposition (native-gesture emulation) is less relevant on Linux desktop than on mobile.
  • Cross-reference: D-26

R-6: Nerd-font glyph icons

  • Rejected: 2026-04-21
  • Reason: TUI hangover from the Python-era clide under legacy/. Not desktop-native; forces a font dependency; doesn't theme consistently. Clide uses custom icon primitives (Tier 6 revisits with proper icon-set design).
  • Cross-reference: Q-17

R-7: CupertinoApp root

  • Rejected: 2026-04-21
  • Reason: iOS-opinionated; wrong shell for a Linux-primary desktop IDE. Same theming-collision problem as R-3.
  • Cross-reference: D-7

R-8: Riverpod / Provider / BLoC for state

  • Rejected: 2026-04-21
  • Reason: Violates D-31. ChangeNotifier + ListenableBuilder ship in the SDK, fake trivially, and cover the state model we need. The ergonomic wins of Riverpod / Provider don't clear the "new dependency" bar at clide's scale.
  • Cross-reference: D-10

R-9: Port planning tooling into clide

  • Rejected: 2026-04-21
  • Reason: Earlier in the planning session the assumption was "clide owns Dart subcommands for decisions + tickets." That breaks the day a contributor works in a terminal or in VS Code / JetBrains — they have no clide binary to run. Reversing: pql owns planning long-term (see D-39); clide consumes via shell-out.
  • Cross-reference: D-39

R-10: Python-script stopgap under tooling/db/

  • Rejected: 2026-04-21
  • Reason: Location, not language. Settled-reach puts scripts at tooling/db/ — copying that path here creates a script-pollution problem: every project using the pattern commits its own copy. The accepted Python port (D-40) lives at tools/scripts/plan, clearly signalled as dev-tooling and time-limited.
  • Cross-reference: D-40

R-11: Permanent stopgap

  • Rejected: 2026-04-21
  • Reason: If the Python port under tools/scripts/plan outlasts pql's feature parity, delete it. The deletion commit should be one changeset: remove tools/scripts/plan, remove its Makefile target (decisions-validate rewires to pql decisions validate), add a CHANGELOG.md entry under Removed, and verify .pql/pql.db still opens under the new pql binary.
  • Cross-reference: D-40

R-12: MaterialApp wrapper from design handoff

  • Rejected: 2026-04-22
  • Reason: The design handoff delivers theme files as MaterialApp/ThemeData Dart classes. This is the delivery format of claude.ai/design, not a design intent. Adopting Material's widget system would contradict D-7 (bare WidgetsApp, no Material/Cupertino). We translate the palette tokens and syntax roles into our existing YAML + SurfaceTokens pipeline.
  • Cross-reference: D-43
Reference in New Issue View Git Blame Copy Permalink