mirror of
https://github.com/priyanshujain/sanderling.git
synced 2026-10-02 11:07:10 +00:00
* feat(hierarchy): name the route a native tree shows The screen name was web-only: the Chrome driver stamps sanderling-screen on the root and nothing else does, so every Android and iOS step recorded and logged an empty screen. The route marker the tree already carries (the resource id ending in Screen, the same one Transitional counts) names it. * feat(runner): say what each step did in the step log One line per step carried only an index and a node count. It now names the screen, the action, its target and the typed value, the last through the same redaction the trace and the prompt use. Emitted after the apply so the line reports what actually happened, skip reason included. * fix(sidecar): state on android whether a field is a secure entry maestro's tree mapper copies a fixed attribute list off the device's XML and password is not on it, so no android element ever reported the fact and the conservative rule downstream redacted every typed value in the trace, the prompt and the log. The XML still carries it: re-read it once per settled snapshot and state the fact on the text fields it matches. A field it cannot match stays unstated, which still reads as a credential. * docs: correct the record that android never reports a secure field Four places said android reports the fact for nothing and that every typed value there is redacted. The sidecar now states it, so they described the old behaviour. * test(sidecar): fail the build if maestro renames the call the fact comes from * fix(sidecar): state the fact on a field named by its hint alone collectTextFields matched on class only, so a node the go side calls editable off its hintText was left unstated and its typed value redacted. * docs: record that ios and web state secure:false for compose password fields Both derive the fact from a widget type a compose app never has, so the value reaches the trace in the clear. Verified on folio on both targets. * feat(android): read the application id out of an apk parses the compiled AndroidManifest.xml rather than shelling out to aapt2, which lives in the versioned build-tools directory that hosts with only platform-tools never install. Claude-Session: https://claude.ai/code/session_012PVErdr3ZzyUASeVQDWsUc * feat(cli): let --android-app-path supply the bundle id --bundle-id stays required everywhere else, and an explicit one still wins, so the apk can never quietly override what was asked for. Claude-Session: https://claude.ai/code/session_012PVErdr3ZzyUASeVQDWsUc * docs: record that the apk can name the package itself Claude-Session: https://claude.ai/code/session_012PVErdr3ZzyUASeVQDWsUc * fix(testrun): pass the jvm the flag that silences the jdk 24 unsafe warning * feat(folio): ask which android device to run on when none is named * feat(folio): pin ios recipes to one simulator udid and ask when several match * docs(ci): say how just ios lands on the simulator the boot step chose * chore(folio): ignore run output anywhere under examples/folio * refactor(hierarchy): name no screen for a tree Transitional calls a cross-fade ScreenName kept its own reading of the route markers and disagreed with Transitional on a marker repeated by a nested node: it named the screen on a step the runner was skipping as unsettled. One reading now. * refactor(sidecar): inline the one attempt passed to callViewHierarchy A named constant and its own comment for a literal used once. * test(sidecar): compare the whole tree when checking the annotation changes nothing else The old assertions checked one id string and one bounds value, and passed with every other attribute stripped off every node. Now the annotated tree minus the two facts it stated must equal the input. * fix(sidecar): match a field to its xml node by class as well as id and bounds A wrapper drawn to the same bounds as the untagged field inside it shared the field's key, both were dropped as ambiguous, and every value typed into an untagged field was redacted. The class tells them apart. * docs(runs): record what the android hierarchy re-read costs per step Two 1m runs per binary on folio, same seed, before and after the re-read.
90 lines
3.6 KiB
Go
90 lines
3.6 KiB
Go
package verifier
|
|
|
|
import (
|
|
"github.com/dop251/goja"
|
|
|
|
"github.com/priyanshujain/sanderling/internal/hierarchy"
|
|
)
|
|
|
|
// RedactedInputText stands in for a typed value in every record. It is fixed,
|
|
// so it gives away neither the value nor its length.
|
|
const RedactedInputText = "[redacted]"
|
|
|
|
// secureFact is what a target says about being a secure text entry. `reported`
|
|
// separates "the platform says this is not one" from "the platform says
|
|
// nothing", which are the two cases the redaction rule has to tell apart.
|
|
type secureFact struct {
|
|
reported bool
|
|
secure bool
|
|
}
|
|
|
|
// secureFactFromHandle reads an element handle's own report. The web host
|
|
// injects handles built in the page, which carry no selector to resolve against
|
|
// the goja-side tree, so the handle is the only thing that knows.
|
|
func secureFactFromHandle(object *goja.Object) secureFact {
|
|
value := object.Get("secure")
|
|
if value == nil || goja.IsUndefined(value) || goja.IsNull(value) {
|
|
return secureFact{}
|
|
}
|
|
return secureFact{reported: true, secure: value.ToBoolean()}
|
|
}
|
|
|
|
func secureFactOf(element *hierarchy.Element) secureFact {
|
|
if element == nil {
|
|
return secureFact{}
|
|
}
|
|
return secureFact{reported: element.SecureReported(), secure: element.Secure}
|
|
}
|
|
|
|
// RecordedActionText renders the typed value of an action for anything that is
|
|
// persisted or sent, resolving the action's target in the tree it was chosen
|
|
// against.
|
|
func RecordedActionText(action Action, tree *hierarchy.Tree) string {
|
|
if action.Kind != ActionKindInputText {
|
|
return action.Text
|
|
}
|
|
var target *hierarchy.Element
|
|
if tree != nil && action.On != "" {
|
|
target = tree.Find(action.On)
|
|
}
|
|
return recordedInputText(action.Text, secureFactOf(target))
|
|
}
|
|
|
|
// RecordedAction is the action as everything downstream of the dispatch sees
|
|
// it. The runner reports the previous step's action to the spec as
|
|
// state.lastAction, and a spec extracting it (examples/folio/sanderling/spec.ts)
|
|
// writes it to the trace, so the copy the runner keeps carries the recorded
|
|
// text rather than the typed one.
|
|
//
|
|
// The decision is made here rather than where the two hosts render
|
|
// state.lastAction because only the caller holds the tree that can answer it.
|
|
// The hosts hold the NEXT step's tree, where the selector may name a different
|
|
// element (a revealed password field reports secure:false) or none at all, and
|
|
// cmd/internal-tools/oracle-reduction replays state.lastAction from the trace's
|
|
// already-recorded text, which redacting here matches exactly.
|
|
func RecordedAction(action Action, tree *hierarchy.Tree) Action {
|
|
action.Text = RecordedActionText(action, tree)
|
|
return action
|
|
}
|
|
|
|
// recordedInputText is the one place a typed value is rendered for a record.
|
|
// The prompt's recent-action memory, the numbered candidate list, the trace and
|
|
// the action the runner reports back as state.lastAction all go through it, so
|
|
// a fifth record added later cannot publish a value the other four withhold.
|
|
// The driver dispatch reads Action.Text directly and is the only reader of the
|
|
// real value, which is what keeps the app receiving the keystrokes a user would
|
|
// have produced.
|
|
//
|
|
// A target the platform reports as a secure entry is redacted, and so is a
|
|
// target carrying no report at all. All three platforms state the fact on their
|
|
// editable elements, so a missing one is a field none of them could speak for:
|
|
// an action that named no target, or an Android text field the sidecar could
|
|
// not match against the device's own view hierarchy. The target that cannot be
|
|
// told apart is treated as the credential.
|
|
func recordedInputText(text string, target secureFact) string {
|
|
if target.reported && !target.secure {
|
|
return text
|
|
}
|
|
return RedactedInputText
|
|
}
|