Skip to content

docs(mutation): state the file-name rule that marks test support code - #4011

Merged
thymikee merged 1 commit into
mainfrom
docs/mutation-support-module-rule
Oct 28, 2026
Merged

thymikee merged 1 commit into
mainfrom
docs/mutation-support-module-rule

Conversation

@thymikee

@thymikee thymikee commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Summary

Follow-up to #3994 review (comment). The isTestSupportModule comment said shared packages stay opaque. The rule reads only the file name, so a *-fixtures subpath that a workspace package exports is looked through as well. @agent-device/test-kit stays opaque only because tests reach it through its barrel. The comment now states that rule. Comment only; no behavior change.

Validation

  • pnpm check:affected --run --head 58056ff752: passed, exactCommit=true.
  • Not device-facing.

🤖 Generated with Claude Code

View guided diff Turn on auto-fix

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
agent-device-website Error Error Oct 28, 2026 11:24am UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

View guided diff | Turn on auto-fix | Re-trigger cubic

@github-actions

Copy link
Copy Markdown
Contributor

Size Report

Metric Base Current Diff
Installed (including dependencies) 5.56 MB 5.56 MB +79 B
Package (unpacked) 5.56 MB 5.56 MB +79 B
Package (download) 1.69 MB 1.69 MB +30 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 38.2 ms 38.9 ms +0.7 ms
CLI --help 78.1 ms 78.8 ms +0.6 ms

@thymikee

Copy link
Copy Markdown
Member Author

The change looks good at 58056ff. The change only edits a comment in scripts/mutation/ownership.ts, and the file-name rule it states reads correctly. The failing Vercel check builds the website, which does not read this file, so it is likely unrelated. I did not read the Vercel failure log. Not blocking: the example at https://github.com/callstack/agent-device/blob/58056ff/scripts/mutation/ownership.ts#L134 names @agent-device/capture-kit/audio-probe-descriptor-fixtures, which is one of the package-export-surface offenders that may only shrink (see scripts/layering/package-export-surface.ts). You could drop the named subpath and keep the pattern, for example "a *-fixtures subpath a workspace package still exports". You can take it or leave it.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Oct 28, 2026
@thymikee
thymikee merged commit ad844ca into main Oct 28, 2026
20 of 21 checks passed
@thymikee
thymikee deleted the docs/mutation-support-module-rule branch October 28, 2026 12:50
@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-28 12:56 UTC

thymikee added a commit that referenced this pull request Oct 28, 2026
* origin/main: (76 commits)
  fix(open): refuse --launch-console on a provider-owned device before the relaunch close (#3938)
  fix(test-app): keep the inert surface control above the settings fold (#3993)
  fix(fill): redact the read-back of a sensitive fill (#4014)
  fix(apple): read the runner cache toolchain key from Xcode's own files (#3995)
  docs: point stable test suites to e2e (#4010)
  docs(mutation): state the file-name rule that marks test support code (#4011)
  refactor(kernel): read the Action Button leaf from isHandheldAppleDevice (#3988)
  fix(ios): match alert labels through the OS prompt strings of the preferred languages (#4006)
  fix(mutation): measure each kernel against the tests that import it (#3994)
  fix(webdriver): avoid unused pre-tap focus geometry reads (#3997)
  refactor(apple): share coordinate drag execution (#4003)
  fix(apple): type a text-entry-trait element as a text input in the simulator AX tree (#4008)
  refactor(contracts): own the unsupported-interactor stub both platforms duplicated (#4002)
  refactor: reuse canonical device identity projections (#3996)
  refactor(apple): share simulator status-bar diagnostics (#3998)
  refactor(runtime): share host command runner construction (#3999)
  refactor(android): centralize session error encoding (#4000)
  chore: mark 0.21.33 as released
  chore(gates): detect new native duplication with jscpd (#3980)
  fix(android): publish the Compose role on its clickable parent instead of a fake child (#3983)
  ...
thymikee added a commit to janjankowski-cs/agent-device that referenced this pull request Oct 28, 2026
* base-3865: (76 commits)
  fix(open): refuse --launch-console on a provider-owned device before the relaunch close (callstack#3938)
  fix(test-app): keep the inert surface control above the settings fold (callstack#3993)
  fix(fill): redact the read-back of a sensitive fill (callstack#4014)
  fix(apple): read the runner cache toolchain key from Xcode's own files (callstack#3995)
  docs: point stable test suites to e2e (callstack#4010)
  docs(mutation): state the file-name rule that marks test support code (callstack#4011)
  refactor(kernel): read the Action Button leaf from isHandheldAppleDevice (callstack#3988)
  fix(ios): match alert labels through the OS prompt strings of the preferred languages (callstack#4006)
  fix(mutation): measure each kernel against the tests that import it (callstack#3994)
  fix(webdriver): avoid unused pre-tap focus geometry reads (callstack#3997)
  refactor(apple): share coordinate drag execution (callstack#4003)
  fix(apple): type a text-entry-trait element as a text input in the simulator AX tree (callstack#4008)
  refactor(contracts): own the unsupported-interactor stub both platforms duplicated (callstack#4002)
  refactor: reuse canonical device identity projections (callstack#3996)
  refactor(apple): share simulator status-bar diagnostics (callstack#3998)
  refactor(runtime): share host command runner construction (callstack#3999)
  refactor(android): centralize session error encoding (callstack#4000)
  chore: mark 0.21.33 as released
  chore(gates): detect new native duplication with jscpd (callstack#3980)
  fix(android): publish the Compose role on its clickable parent instead of a fake child (callstack#3983)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant