ccqa 1.40.5 → 1.40.7

package/dist/bin/ccqa.mjs

@@ -1660,7 +1660,7 @@ async function invokeClaudeStreaming(options, onEvent) {
 					};
 					if (hasRefSelector(cmd)) return {
 						decision: "block",
-						reason: "@ref selectors (like @e14) are session-specific and change every run. They cannot be used in generated tests. Use one of the allowed selector formats instead: [aria-label='...'], text=..., [placeholder='...'], or [type='password']. Take a fresh snapshot and find the element's aria-label or visible text."
+						reason: "@ref selectors (like @e14) are session-specific and change every run. They cannot be used in generated tests. Use one of the allowed selector formats instead: [aria-label='...'], text=..., [placeholder='...'], or [type='password']. Take a fresh snapshot and find the element's aria-label or visible text. If an allowed selector already clicks the element but nothing happens, the element is clipped by an inner scroll container: `scrollintoview` it (addressed by a CSS selector, not `text=`) and click again — a @ref would not have fixed that either."
 					};
 					const bareTag = findPositionalBareTag(cmd);
 					if (bareTag !== null) return {
@@ -15289,12 +15289,18 @@ function assertToLine(action, locator) {
 	return assertLine ?? comment;
 }
 /**
-* `url_contains` → toHaveURL. Literal values become an unanchored RegExp
-* (substring match); values carrying env refs can't live in a regex literal,
-* so they use toHaveURL's glob form with a template literal.
+* `url_contains` → "the URL contains this".
+*
+* A `${VAR}` only has a value at run time, which rules out both forms that take
+* the pattern up front: a regular expression would have to match the reference
+* span with `.*` (so `${APP_BASE_URL}` alone would assert nothing at all), and
+* the glob `toHaveURL` accepts is compared against the whole URL, so an
+* absolute one never matches. Polling `page.url()` keeps the substring
+* semantic and the resolved value.
 */
 function urlContainsAssert(value) {
-	if (jExpr(value).startsWith("`")) return `await expect(page).toHaveURL(${envRefsToJsExpression(`**${value}**`)});`;
+	const expr = jExpr(value);
+	if (expr.startsWith("`")) return `await expect.poll(() => page.url()).toContain(${expr});`;
 	return `await expect(page).toHaveURL(new RegExp(${j(escapeRegExp(value))}));`;
 }
 function escapeRegExp(s) {
@@ -17829,6 +17835,7 @@ find nth <index> "<ALLOWED-css>" <action>
 5. For checkboxes: try \`check "text=Label"\` or \`check "[aria-label='Label']"\`.
 6. If repeated labels make every ALLOWED selector ambiguous → use the \`find\` subset above.
 7. Never guess. If a selector fails, take a fresh snapshot before retrying.
+8. **A click that reports success but changes nothing means the element sits outside the visible part of an inner scroll container** — the event dispatches, the app never reacts, and no error is printed. Run \`scrollintoview\` on the element and click again. Scroll to the element itself, addressed by a CSS selector (\`scrollintoview "button.primary-action"\`); \`scrollintoview "text=…"\` can land on a text node that leaves the control still clipped. Do not read this as a product bug or reach for a \`@ref\` — verify by checking that the DOM changed (e.g. \`get count\` on what the click should have produced).
 
 ### Special input types
 

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ccqa",
-  "version": "1.40.5",
+  "version": "1.40.7",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ccqa",
-  "version": "1.40.5",
+  "version": "1.40.7",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {