Skip to content

docs: delete redundant docs and trim README - #1956

Merged
benceruleanlu merged 2 commits into
bl/desktop-as-a-platformfrom
bl/desktop-test-architecture-audit-c77650
Nov 6, 2026
Merged

benceruleanlu merged 2 commits into
bl/desktop-as-a-platformfrom
bl/desktop-test-architecture-audit-c77650

Conversation

@benceruleanlu

@benceruleanlu benceruleanlu commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Deletes docs that mostly repeat what's already in the code:

  • TESTING.md and e2e/README.md restated package.json, the Vitest and Playwright configs, and the CI workflows. The e2e spec list was out of date: it was missing 19 of the 59 specs.
  • docs/instance-navigation-926-handoff.md was a handoff note for [Refactor] Tech Debt #926.

Trims README.md:

  • Removes the scripts table, the project tree and the release workflow table, which repeated package.json, the source folders and .github/workflows.
  • Keeps setup, bootstrap Python, the release steps, the secrets list, data locations and reset steps.
  • Removes the links to the deleted files.

Most of both files restated package.json scripts, the Vitest and
Playwright configs, and the CI workflows, and the e2e spec inventory
had drifted (19 of 59 specs missing). Drop the README links to them.
@benceruleanlu
benceruleanlu requested review from a team as code owners November 6, 2026 11:22
@coderabbitai

coderabbitai Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cdebbb6a-dc54-4fa1-a674-85b4b01f1167

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Drop the README sections that restated package.json scripts, the
source tree, and the release workflow files.
@benceruleanlu benceruleanlu changed the title docs: delete TESTING.md and e2e/README.md docs: delete redundant docs and trim README Nov 6, 2026
@benceruleanlu
benceruleanlu merged commit d730211 into bl/desktop-as-a-platform Nov 6, 2026
16 checks passed
@benceruleanlu
benceruleanlu deleted the bl/desktop-test-architecture-audit-c77650 branch November 6, 2026 12:03
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.

1 participant