Skip to content

Release v0.1.0: Review-Korrekturen, CI und Release-Workflow - #2

Merged
Kyloschmeilo merged 3 commits into
mainfrom
claude/epic-brahmagupta-fnrowp
Oct 11, 2026
Merged

Release v0.1.0: Review-Korrekturen, CI und Release-Workflow#2
Kyloschmeilo merged 3 commits into
mainfrom
claude/epic-brahmagupta-fnrowp

Conversation

@Kyloschmeilo

@Kyloschmeilo Kyloschmeilo commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Bereitet das erste Release v0.1.0 vor. Folgt auf den bereits gemergten PR #1.

Inhalt

  • Korrekturen aus einer mehrstufigen Code-Review: fünf Bereiche, jeder Fund unabhängig gegengeprüft; 37 bestätigte Funde, alle behoben.
    • Eingaben (cdp-agent.js):
      • Enter löst keypress aus, ein abgebrochenes keypress verhindert das Absenden; Enter auf Submit-Inputs/<summary> aktiviert das fokussierte Element
      • macOS-Tastenbefehle
      • keine Klicks auf deaktivierte Elemente
      • Dreifachklick, Strg+Pos1/Ende
      • Tab-Reihenfolge inkl. Shadow DOM
      • Pop-ups (stopPropagation, Formulare, noopener, benannte Frames)
      • Escape für Dialoge/Popover, Pfeiltasten in Zahlenfeldern und Radio-Gruppen
      • Console-Serialisierung ohne Getter
    • Hintergrund (engine.js):
      • Event-Page bleibt wach, solange ein Tab angehängt ist
      • fremde iframes überschreiben nicht mehr Console-/Netzwerklisten
      • Screenshots bei Seiten-Zoom haben die richtige Größe
      • await funktioniert auch mit zone.js
      • die Console-Registrierung blockiert keine Befehle mehr
    • Sicherheit (background.js): abgefangene OAuth-Weiterleitungen nur mit passendem oauthState; keine Extension-Seiten mehr in Web-Tabs.
    • Build/Windows: Symlinks/Junctions, ZIP-Ordner mit Backslash, Sandbox-Seiten, PowerShell liest UTF-8.
    • Doku:
      • Update-Anleitung
      • neue Grenzen: Zwischenablage, „Seite verlassen?“-Dialoge
      • Abschnitt zum Selbst-Signieren entfernt, weil er Anthropics Code weitergäbe
      • LICENSE wieder wortgleich MIT
  • CI (.github/workflows/ci.yml): Unit-Tests und Syntaxprüfung unter Linux und Windows, Node 18 und 22.
  • Release (.github/workflows/release.yml): Ein Tag v* erstellt ein GitHub-Release mit den Notizen aus CHANGELOG.md, ohne gebaute .xpi (die enthielte Anthropic-Code).
  • CHANGELOG.md mit den Notizen für 0.1.0.

Teststand

  • 19/19 Unit-Tests, 52/52 End-to-End-Prüfungen in Firefox 157 (gegen die ältere Extension-Version 1.0.70), darunter neue Regressionstests für Enter/keypress, deaktivierte Buttons, Dreifachklick und abgelehnte fremde OAuth-Weiterleitungen.
  • web-ext lint: 0 Fehler.
  • Nicht getestet: die aktuelle Extension-Version, die Anmeldung mit echtem Konto, die Windows-Skripte unter echtem Windows (die CI führt die Unit-Tests unter Windows aus), Claude Code / Desktop.

Nach dem Merge: Tag v0.1.0 auf main → GitHub Actions erstellt das Release.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BpfwyChUnZawLQokRG5xQT

Generated by Claude Code

claude added 3 commits October 11, 2026 18:51
@claude
CI-Tests (Linux/Windows, Node 18/22), Release-Workflow und Changelog
e40920a 
- .github/workflows/ci.yml: Unit-Tests und Syntaxprüfung der Shims bei jedem Push/PR
- .github/workflows/release.yml: erstellt bei einem Tag v* ein GitHub-Release mit den
  Notizen aus CHANGELOG.md (ohne gebaute .xpi, da diese Anthropic-Code enthielte)
- CHANGELOG.md: Notizen für 0.1.0

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BpfwyChUnZawLQokRG5xQT
@claude
Merge remote-tracking branch 'origin/main' into claude/epic-brahmagup…
98c4369 
…ta-fnrowp
@claude
Korrekturen aus der Review vor dem Release v0.1.0
897d62c 
Eingaben (cdp-agent.js):
- Enter löst wie in Chrome keypress (13) aus; ein abgebrochenes keypress verhindert
  Absenden/Zeilenumbruch. Enter auf Submit-/Button-Inputs und <summary> aktiviert das
  fokussierte Element statt des ersten Submit-Buttons.
- macOS-Tastenbefehle: fehlende Befehle ergänzt, unbekannte fallen auf die normale
  Tastenaktion zurück.
- Keine Klick-Events auf deaktivierte Formular-Elemente; kein Klick ohne vorheriges
  Drücken im selben Dokument.
- Dreifachklick und Strg+Pos1/Ende in Editoren (Firefox kennt die Granularitäten nicht).
- Tab überspringt versteckte/deaktivierte Elemente und berücksichtigt Shadow DOM.
- Pop-ups: stopPropagation, Formulare mit target=_blank, noopener, benannte Frames.
- Escape schließt modale Dialoge/Popover; Pfeiltasten in Zahlenfeldern und
  Radio-Gruppen; Tastatur-Scrollen in inneren Scrollbereichen.
- Console: kein Ausführen von Getters, begrenzte Serialisierung, Meldungen vor einer
  Navigation gehen nicht verloren.

Hintergrund (engine.js):
- Event-Page bleibt wach, solange ein Tab angehängt ist (sonst ginge der Zustand verloren).
- Console/Netzwerk aus fremden iframes überschreiben nicht mehr die Listen der Seite.
- Screenshots bei Seiten-Zoom haben die richtige Größe.
- JavaScript mit await funktioniert auch auf Seiten mit zone.js.
- Console-Registrierung blockiert die Befehlswarteschlange nicht mehr.

Sicherheit (background.js):
- Abgefangene OAuth-Weiterleitungen werden nur mit passendem oauthState verarbeitet;
  Extension-Seiten werden dafür nicht mehr in Web-Tabs geöffnet.

Build/Windows: Erkennung als Hauptskript über Symlinks/Junctions, ZIP-Ordnereinträge
mit Backslash, Sandbox-Seiten bleiben erhalten, PowerShell liest UTF-8 korrekt.

Doku: Update-Anleitung, Grenzen (Zwischenablage, „Seite verlassen?“-Dialoge),
Tastenkürzel bei temporärem Add-on, Abschnitt zum Selbst-Signieren entfernt
(Weitergabe von Anthropics Code), LICENSE wieder wortgleich MIT, *.zip ignoriert.

Tests: neue Unit- und End-to-End-Prüfungen (19 Unit-Tests, 52 E2E-Prüfungen grün).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BpfwyChUnZawLQokRG5xQT
@Kyloschmeilo
Kyloschmeilo merged commit 222d2a6 into main Oct 11, 2026
4 checks passed
@Kyloschmeilo Kyloschmeilo mentioned this pull request Oct 11, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kyloschmeilo @claude