aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-22 14:23:34 -0500
committerCraig Jennings <c@cjennings.net>2026-05-22 14:23:34 -0500
commit282a35d30edca8c5ae7a5955111440908413f267 (patch)
treedc58fe16da5b5a2d2b7387b3a727aa0ed0506d41
parent2e9d5b0acb5e6fd13020519e1807d804d5ab8a80 (diff)
downloadrulesets-282a35d30edca8c5ae7a5955111440908413f267.tar.gz
rulesets-282a35d30edca8c5ae7a5955111440908413f267.zip
chore(todo): close playwright headed/headless item
-rw-r--r--todo.org9
1 files changed, 3 insertions, 6 deletions
diff --git a/todo.org b/todo.org
index 200f9b7..00b749f 100644
--- a/todo.org
+++ b/todo.org
@@ -742,7 +742,7 @@ Each item below is a one-line summary of a sub-TODO further down. Tick the box w
**** Browser testing
- [ ] [#A] =playwright-js=: locator/assertion-first guidance (replace raw CSS, =networkidle=)
-- [ ] [#B] =playwright-js= + =playwright-py=: reconcile headless/visible defaults
+- [X] [#B] =playwright-js= + =playwright-py=: reconcile headless/visible defaults
- [ ] [#B] =playwright-js= + =playwright-py=: remove emoji console markers from examples
**** Frontend / UI
@@ -840,12 +840,9 @@ calls =networkidle= discouraged for testing. Keep reconnaissance, but revise it
to wait for a visible app-specific landmark instead of treating network quiet
as readiness.
-*** TODO [#A] =playwright-js= and =playwright-py=: reconcile headless/visible-browser defaults
+*** 2026-05-22 Fri @ 14:23:02 -0500 Added headed/headless decision tables to both playwright skills
-=playwright-js= says visible Chromium by default; =playwright-py= says
-headless by default. That may be intentional, but the difference should be
-explicit: interactive visual debugging -> headed, CI/pytest smoke tests ->
-headless. Add a small decision table so agents don't flip modes by habit.
+Added matching purpose-based decision tables to =playwright-js/SKILL.md= (was "always visible") and =playwright-py/SKILL.md= Best Practices (was "always headless"). Each names its own default and points at the other skill, so the difference is deliberate, not a habit-flip: headed for interactive debugging, headless for CI/pytest. Also softened the absolutist "Always launch... headless" comment in the py example.
*** TODO [#A] =playwright-js= and =playwright-py=: remove emoji console markers from examples