Module refinery.lib.scripts.ps1.deobfuscation.removal
The single route by which a PowerShell cleanup pass removes or rewrites statements in a body.
Expand source code Browse git
"""
The single route by which a PowerShell cleanup pass removes or rewrites statements in a body.
"""
from __future__ import annotations
from typing import NamedTuple
from refinery.lib.scripts import (
BodyEdit,
Node,
Statement,
_replace_in_parent,
owning_field,
owning_list,
reattach,
)
from refinery.lib.scripts.ps1.analysis.effects import fault_is_observed
from refinery.lib.scripts.ps1.model import (
Expression,
Ps1ExpressionStatement,
Ps1TrapStatement,
)
class _Proposal(NamedTuple):
"""
One intended edit: the statement to change, and what stands in its place afterwards.
"""
statement: Statement
replacement: list[Statement]
def _removes_a_handler(statement: Node) -> bool:
"""
Whether the statement *is* fault-handling machinery rather than something that might fault.
A `trap` intercepts a terminating error that would otherwise reach the enclosing `catch`, so
deleting one re-routes the fault although the deleted statement cannot itself raise.
`removals_may_fault` answers only the first half of the fault question — can what this pass
removes throw — and says nothing about this half, so this half is asked of every pass.
"""
return isinstance(statement, Ps1TrapStatement)
def _restore(landed: list[Node]) -> None:
"""
Put the parent pointers inside everything the batch has just left standing back in agreement
with the tree.
Building a replacement adopts the parts of the original it reuses, and `Ps1RemovalPlan.propose`
undoes that at once, so the batch decides against a tree whose pointers are true. Installing one
names its new holder and nothing below it, so what the build claimed has to be claimed again
here.
This runs **after** the whole batch is in place, never between two of its edits. `reattach`
walks a subtree, and until the last edit has landed a replacement's subtree can still contain a
statement another replacement is about to take: repairing then asserts a structure that is not
the final one, and which of the two claims survives is decided by the order the batch happened
to visit them in. Once nothing is left to install, every walk asserts the same structure, and
the order stops mattering.
"""
for node in landed:
reattach(node)
def _fits_a_field(proposal: _Proposal) -> Node | None:
"""
The single node a direct field can take from `proposal`, or `None` when the proposal cannot be
written into one at all.
A field holds what the model declares it holds, and a statement in an `Expression` slot leaves a
tree whose shape contradicts its own declaration: every `unwrap_parens` and every
`isinstance(paren.expression, ...)` gate downstream stops seeing through the parenthesis. The
statement wrapper a pass builds for a body — `refinery.lib.scripts.ps1.model` spells it
`Ps1ExpressionStatement` — carries no meaning a field needs, so it is peeled off rather than
installed.
"""
if len(proposal.replacement) != 1:
return None
replacement = proposal.replacement[0]
if not isinstance(proposal.statement, Expression):
return replacement
if isinstance(replacement, Expression):
return replacement
if isinstance(replacement, Ps1ExpressionStatement):
return replacement.expression
return None
class Ps1RemovalPlan:
"""
A batch of proposed edits to one statement list, committed as a single mutation.
A pass proposes each edit it wants, consults the set-level guards it is responsible for against
`survivors`, and calls `commit`. What the class owns is the part every pass has to get right the
same way: the per-statement veto, and the fact that the whole batch lands as one tree edit.
**Which set a guard must be shown is decided by its polarity, and the two answers are
opposite.**
A guard *permissive* in the survivor set — more survivors means a removal is more likely to be
allowed — must see the **pre-veto** set, which is `survivors`. Both
`refinery.lib.scripts.ps1.analysis.effects.output_is_covered` and
`refinery.lib.scripts.ps1.analysis.effects.pruning_erases_body` are this kind, so a guard shown
the post-veto set would read a vetoed statement as cover for deleting its neighbour, and a
statement kept because a handler observes its fault would license destroying the one beside it.
The veto therefore runs inside `commit`, after every such guard has had its answer, and
`survivors` cannot show one a veto.
A guard *restrictive* in it — more survivors means a removal is **less** likely to be allowed —
must see the post-veto set, or a vetoed statement's dependencies are deleted out from under it.
Reachability and liveness are this kind, and they read `accepted`, which reports what this plan
would edit rather than reconstituting a survivor set the permissive guards could reach.
Set-level guards deliberately stay with the passes. They are not the same from one pass to the
next — `Ps1DeadCodeElimination` declines `output_is_covered` as too permissive for what it does,
and `Ps1JunkStatementRemoval._remove_inert_functions` weighs erasure against the definitions
alone rather than against everything it removes — so collapsing them into one call here would
have to be spelled as flags, which is the policy sprawl this class exists to reduce.
`removals_may_fault` is the one thing about a pass this class does have to be told, because the
fault refusal is not the same question for every pass either.
`refinery.lib.scripts.ps1.analysis.effects.fault_is_observed` exists because *nothing* in the
effect substrate can answer whether a statement throws, so a pass that cannot say declines every
removal from a protected `try` body. `Ps1DeadCodeElimination` can say: it removes pure constants
and constructs whose condition it has already proved constant, neither of which can raise. For
such a pass the refusal that remains is the set-level one the per-statement veto was standing in
for — the batch must not leave the protected body empty, because an empty `try` block is
evidence about the pass rather than about the code as written.
`all_or_nothing` is the second, for a batch whose parts are one edit rather than several. A veto
normally skips the proposal it lands on and lets the rest through, which is right when each
stands alone; it is wrong when a partly applied batch is not a smaller edit but a broken one.
`Ps1ControlFlowDeflattening` replaces a dispatcher loop and deletes the `$state = ...` seeding
it, and applying either alone leaves a state machine half dissolved. Neither the class nor the
veto can tell the two cases apart, so the pass says which it is.
"""
def __init__(
self,
parent: Node,
attr: str = 'body',
removals_may_fault: bool = True,
all_or_nothing: bool = False,
):
self.parent = parent
self.attr = attr
self.removals_may_fault = removals_may_fault
self.all_or_nothing = all_or_nothing
self._proposals: dict[int, _Proposal] = {}
def propose(
self,
statement: Statement,
replacement: list[Statement] | None = None,
) -> None:
"""
Register that `statement` is to be replaced by `replacement`, or removed when that is `None`
or empty. Proposing the same statement twice keeps the later proposal.
A replacement is not a weaker removal: `Ps1DeadCodeElimination` resolves a constant `if`
into the statements of the branch that runs, and a dead store becomes `$Null = <rhs>` so
the value is still computed. A pass that could only delete could express neither.
**A registered replacement holds no claim on the tree until `commit` grants it one.**
Building one adopts the parts of the original it reuses, and the original is still standing,
so the adoption leaves nodes in the tree naming a holder that is not; the statement is put
back in order here, before this call returns. Everything that happens between a proposal and
the verdict reads the tree by walking upward — the set-level guards, the veto, the search
for the list a statement sits in — and a batch that decides against a tree it has already
half detached decides about a tree that does not exist. Making the repair a condition of
registering is also what lets a pass build every replacement up front and withdraw or
abandon afterwards without owing anything.
The repair is owed by every registration, not only by one that ends up carrying a
replacement. What has to be given back is what the *caller* built, and a pass routinely
builds a replacement and then decides against installing it before it ever gets here:
`Ps1DeadCodeElimination` hoists a construct's branch into new statements and only afterwards
drops the ones its set-level guard forbids, which can empty the list. Reading the argument
that arrives as the record of what was built is reading the survivors of that filter, and it
left a statement standing over a literal that named a node the pass had thrown away.
"""
self._proposals[id(statement)] = _Proposal(statement, list(replacement or ()))
reattach(statement)
def withdraw(self, statement: Statement) -> None:
"""
Drop a registered proposal, leaving `statement` where it stands. A pass that shrinks its own
batch after reading `accepted` uses this. Nothing needs putting back, because `propose`
never let the replacement take anything in the first place.
"""
self._proposals.pop(id(statement), None)
def abandon(self) -> None:
"""
Drop every proposal, leaving the tree as it was. Same contract as `withdraw`, for a pass
that built a whole batch and then decided against all of it.
"""
self._proposals.clear()
@property
def survivors(self) -> list[Statement]:
"""
The list as it would stand if every proposal were applied, the veto ignored. This is what
the set-level guards must be shown; see the class docstring for why the post-veto set must
not reach them.
"""
return self._edit(self._proposals.values()).result()
@property
def accepted(self) -> list[Statement]:
"""
The statements `commit` would edit, without editing them.
This is for the *restrictive* guards — the ones that allow **fewer** removals as more
statements survive, which is the opposite polarity to `survivors`' readers. Reachability is
the example: it concludes a function is dead from the call sites that are going away, so a
vetoed caller it never heard about leaves the emitted script calling a function it does not
define. Such a guard asks this, drops what it now forbids with `withdraw`, and asks again;
the loop terminates because the batch only shrinks.
**That last part is a fact about the default flags, not about this query.** Under
`all_or_nothing`, and under `removals_may_fault=False` against a protected body, a
withdrawal can take this set from empty to non-empty — the batch *grows*, and a loop resting
on the shrinking argument does not terminate. `Ps1RemovalPlans` opens every plan with the
defaults, which is why its consumers may write that loop; a pass that sets either flag and
then loops on `accepted` owes its own termination argument.
What a caller obtains is what *this* plan would do, not a post-veto survivor set. The
distinction is the whole safety argument: `survivors` still cannot show a permissive guard a
veto, so a statement the veto keeps never becomes licence to delete the one beside it.
This must not edit the tree. A query that installs a replacement's claim on its children is
a query that decides the batch, and the guard asking it has not decided anything yet; the
claims are granted in `commit`, where the verdict is final.
A caller may read this as exact, and every consumer does: membership means *this is going
away*, so a statement reported here that `commit` then leaves standing is a rescue that
never happens and a dependency deleted out from under it — the failure the restrictive
polarity exists to prevent, not the safe side of it. The verdict a plan gives here is the
one it applies, and `Ps1RemovalPlans` reaches every verdict before it lands the first edit
so that stays true across a batch.
What that exactness rests on is the plan's list actually holding what was proposed against
it, which is a fact about the proposal and not one this query can establish: `BodyEdit`
ignores a splice for a node its list does not hold, and `_apply` reports such a proposal as
landing nothing. `Ps1RemovalPlans.propose` establishes it by finding the list, and
`Ps1RemovalPlans.propose_in` moves it to the caller, which is what a caller of that method
takes on.
"""
return [proposal.statement for proposal in self._allowed()]
def _edit(self, proposals) -> BodyEdit:
edit = BodyEdit(self.parent, self.attr)
for proposal in proposals:
edit.splice(proposal.statement, proposal.replacement)
return edit
def _vetoed(self, proposal: _Proposal) -> bool:
"""
Whether a single proposal must be skipped although the guards allowed the batch.
`fault_is_observed` asks whether the edit changes which code runs, and it is asked of
deletions only, because a rewrite that keeps evaluating the original expression still throws
where the original threw, so an enclosing handler stays as reachable as it was. It is asked
only of a pass that cannot rule the fault out itself — or of a statement that *is* a
handler, which no pass can rule out; see `_removes_a_handler`.
"""
if proposal.replacement:
return False
if not (self.removals_may_fault or _removes_a_handler(proposal.statement)):
return False
return fault_is_observed(proposal.statement)
def _allowed(self) -> list[_Proposal]:
"""
The proposals that survive the veto and every set-level refusal this class owns. Kept
apart from `commit` so that `accepted` is a query over the decision rather than a second
copy of it.
"""
proposals = list(self._proposals.values())
allowed = [p for p in proposals if not self._vetoed(p)]
if self.all_or_nothing and len(allowed) != len(proposals):
allowed = []
if self._empties_a_protected_body(allowed):
allowed = []
return allowed
def commit(self) -> bool:
"""
Apply every proposal no veto blocks, as one edit, and report whether the tree moved.
"""
moved, landed = self._apply(self._allowed())
_restore(landed)
return moved
def _apply(self, allowed: list[_Proposal]) -> tuple[bool, list[Statement]]:
"""
Land one already-reached verdict, reporting whether the tree moved and which nodes the list
now holds because of it. Split out for `Ps1RemovalPlans`, which has to reach every verdict
before it lands any of them, and land every edit before it repairs any of them.
What landed is decided per splice rather than taken from `allowed`, because the two are not
the same set. `BodyEdit` ignores a splice for a node its list does not hold — the class
describes an edit to one list and nothing else — so a replacement can be allowed and still
never be installed. Repairing that one would hand it the children it is still not holding,
which is the corruption `propose` undoes at registration, reintroduced by the repair.
The question a splice was honoured is asked of the statement it names and not of the
resulting list, and only the first is the same question: a replacement that already stands
in the list is carried over by an edit that ignored its splice, so reading the result back
reports it installed by an edit that installed nothing.
"""
if not allowed:
return False, []
held = {id(item) for item in getattr(self.parent, self.attr, None) or []}
if not self._edit(allowed).apply():
return False, []
return True, [
statement
for proposal in allowed
if id(proposal.statement) in held
for statement in proposal.replacement
]
def _empties_a_protected_body(self, allowed: list[_Proposal]) -> bool:
"""
Whether committing `allowed` would clear a `try` body beside a handler that acts. Asked only
of a pass that ruled the per-statement fault question out; for every other pass the veto has
already declined each such removal one at a time. The emptiness test lives here rather than
at the call site so that the two halves of the name are decided in one place, and so that
the `BodyEdit` it needs is built only for the pass that can use the answer.
"""
if self.removals_may_fault or not allowed:
return False
if self._edit(allowed).result():
return False
return any(fault_is_observed(proposal.statement) for proposal in allowed)
class Ps1RemovalPlans:
"""
One `Ps1RemovalPlan` per statement list, for a pass that finds its removals by walking the whole
tree rather than by descending body by body. Each list still commits as a single edit, so a pass
scattering removals across a script advances the mutation counter once per body it touches
instead of once per statement.
A whole-tree walk also reaches statements that sit in no list at all: the inner store of
`($y = ($z = 1))` is a statement to every pass that finds it and a direct field to its parent.
Those are carried here too, because a pass that finds one has no other route left — but only as
rewrites. A field cannot lose its statement without the parent losing its shape, so a proposal
to remove one outright is registered and then declined at commit, which is also why the fault
veto has nothing to say about them: it declines deletions, and none of these is one.
"""
def __init__(self):
self._plans: dict[tuple[int, str], Ps1RemovalPlan] = {}
self._rewrites: dict[int, _Proposal] = {}
#: The filed statement is kept beside its plan, and not only its `id`, for the reason
#: `refinery.lib.scripts.BodyEdit` keeps a spliced node beside its own: a statement that is
#: collected while its entry stands would let the next object at that address be withdrawn
#: from a plan that never held it.
self._filed: dict[int, tuple[Statement, Ps1RemovalPlan]] = {}
def propose_in(
self,
parent: Node,
statement: Statement,
replacement: list[Statement] | None = None,
attr: str = 'body',
) -> None:
"""
Register an edit against the list `parent.<attr>`, which the caller states holds
`statement`.
`propose` has to find that list, and finding it is an identity scan over the list — the cost
of one proposal is the length of the body, so the cost of a pass is the square of it. A pass
that walks bodies to find its removals is already holding the list, and says so here.
Nothing checks the claim, so a caller that names the wrong list files a proposal `commit`
will silently drop and `accepted` will still report; see `Ps1RemovalPlan.accepted` for what
rests on it. Filing the same statement a second time drops the first proposal rather than
leaving it standing, because the alternative is a statement `withdraw` can only reach one of
— a withdrawal that half happens is what remembering where a proposal landed exists to
rule out.
"""
plan = self._plan_for(parent, attr)
filed = self._filed.get(id(statement))
if filed is not None and filed[1] is not plan:
filed[1].withdraw(statement)
plan.propose(statement, replacement)
self._filed[id(statement)] = (statement, plan)
def _plan_for(self, parent: Node, attr: str) -> Ps1RemovalPlan:
key = (id(parent), attr)
try:
return self._plans[key]
except KeyError:
plan = self._plans[key] = Ps1RemovalPlan(parent, attr)
return plan
def propose(
self,
statement: Statement,
replacement: list[Statement] | None = None,
) -> bool:
"""
Register an edit with the plan for the list holding `statement`, opening one if this is the
first edit against that list, or as a direct-field rewrite when `statement` sits in no list.
Reports whether the statement can be edited at all.
A refusal releases the proposal rather than handing it back: the caller has already built
its replacement, and building one adopts parts of the statement, so a refusal the caller has
to remember to undo is a refusal that gets forgotten. See `Ps1RemovalPlan.propose` for why
no registered replacement holds a claim before `commit` either, and for why the release is
owed whatever the `replacement` argument turns out to hold.
"""
owner = owning_list(statement)
if owner is None:
if owning_field(statement) is None:
reattach(statement)
return False
self._rewrites[id(statement)] = _Proposal(statement, list(replacement or ()))
reattach(statement)
return True
parent, attr = owner
self.propose_in(parent, statement, replacement, attr)
return True
def withdraw(self, statement: Statement) -> None:
"""
Drop a registered proposal wherever it landed. Same contract as `Ps1RemovalPlan.withdraw`.
Where it landed is remembered rather than looked up again. Rediscovering the owning list
reports nothing when it fails, and a withdrawal that quietly does not happen is a proposal
the caller has already written off and `commit` still applies — half of a group edit whose
other half is gone.
"""
if self._rewrites.pop(id(statement), None) is not None:
return
filed = self._filed.pop(id(statement), None)
if filed is not None:
filed[1].withdraw(statement)
def abandon(self) -> None:
"""
Drop every proposal in every plan. Same contract as `Ps1RemovalPlan.abandon`.
"""
for plan in self._plans.values():
plan.abandon()
self._rewrites.clear()
self._filed.clear()
def survivors(self, parent: Node, attr: str = 'body') -> list[Statement]:
"""
The pre-veto survivors of one of the lists this batch touches, or its current contents when
no edit was registered against it. Same contract as `Ps1RemovalPlan.survivors`.
"""
plan = self._plans.get((id(parent), attr))
if plan is None:
return list(getattr(parent, attr, None) or [])
return plan.survivors
@property
def accepted(self) -> list[Statement]:
"""
The statements `commit` would edit across every list this batch touches, plus the
direct-field rewrites it would install. Same contract, and the same two limits, as
`Ps1RemovalPlan.accepted`.
A pass that scatters one logical removal across several lists needs this rather than the
per-plan answer: `refinery.lib.scripts.ps1.deobfuscation.unused.Ps1JunkStatementRemoval`
drops an inert definition and the bare calls to it, and those routinely land in different
plans, so a veto on either half is only visible here.
"""
accepted = [statement for plan in self._plans.values() for statement in plan.accepted]
accepted.extend(proposal.statement for proposal, _ in self._installable())
return accepted
def _installable(self) -> list[tuple[_Proposal, Node]]:
"""
The direct-field rewrites this batch would install, each beside the node the field takes.
One decision, read by `accepted` and applied by `commit`, so the two cannot drift.
"""
installable = []
for proposal in self._rewrites.values():
replacement = _fits_a_field(proposal)
if replacement is None:
continue
installable.append((proposal, replacement))
return installable
def commit(self) -> bool:
"""
Commit every plan and report whether any of them moved the tree.
Every verdict is reached before the first edit lands, and every edit lands before the first
repair. A veto is a question about the tree —
`refinery.lib.scripts.ps1.analysis.effects.fault_is_observed` reads the handler beside the
body it is asked about — so a plan that emptied a `catch` body would change the answer for
the `try` body's plan, and which plan that is would be decided by nothing better than the
order the batch happened to open them in. That is also what makes `accepted` exact: what it
reported is what commits. `_restore` says why the repairs come last.
Nothing that did not land is repaired. A rewrite the field refused is a replacement that was
released when it was registered and has taken nothing since, so its original owes nothing
either — and asserting an uninstalled statement's structure here is the one walk that could
assert it over a node the tree holds somewhere else.
"""
verdicts = [(plan, plan._allowed()) for plan in self._plans.values()]
rewrites = self._installable()
landed: list[Node] = []
moved = False
for plan, allowed in verdicts:
was_moved, installed = plan._apply(allowed)
moved = moved or was_moved
landed.extend(installed)
for proposal, replacement in rewrites:
if not _replace_in_parent(proposal.statement, replacement):
continue
landed.append(replacement)
moved = True
_restore(landed)
return moved
Classes
class Ps1RemovalPlan (parent, attr='body', removals_may_fault=True, all_or_nothing=False)-
A batch of proposed edits to one statement list, committed as a single mutation.
A pass proposes each edit it wants, consults the set-level guards it is responsible for against
survivors, and callscommit. What the class owns is the part every pass has to get right the same way: the per-statement veto, and the fact that the whole batch lands as one tree edit.Which set a guard must be shown is decided by its polarity, and the two answers are opposite.
A guard permissive in the survivor set — more survivors means a removal is more likely to be allowed — must see the pre-veto set, which is
survivors. Bothrefinery.lib.scripts.ps1.analysis.effects.output_is_coveredandpruning_erases_body()are this kind, so a guard shown the post-veto set would read a vetoed statement as cover for deleting its neighbour, and a statement kept because a handler observes its fault would license destroying the one beside it. The veto therefore runs insidecommit, after every such guard has had its answer, andsurvivorscannot show one a veto.A guard restrictive in it — more survivors means a removal is less likely to be allowed — must see the post-veto set, or a vetoed statement's dependencies are deleted out from under it. Reachability and liveness are this kind, and they read
accepted, which reports what this plan would edit rather than reconstituting a survivor set the permissive guards could reach.Set-level guards deliberately stay with the passes. They are not the same from one pass to the next —
Ps1DeadCodeEliminationdeclinesoutput_is_coveredas too permissive for what it does, andPs1JunkStatementRemoval._remove_inert_functionsweighs erasure against the definitions alone rather than against everything it removes — so collapsing them into one call here would have to be spelled as flags, which is the policy sprawl this class exists to reduce.removals_may_faultis the one thing about a pass this class does have to be told, because the fault refusal is not the same question for every pass either.fault_is_observed()exists because nothing in the effect substrate can answer whether a statement throws, so a pass that cannot say declines every removal from a protectedtrybody.Ps1DeadCodeEliminationcan say: it removes pure constants and constructs whose condition it has already proved constant, neither of which can raise. For such a pass the refusal that remains is the set-level one the per-statement veto was standing in for — the batch must not leave the protected body empty, because an emptytryblock is evidence about the pass rather than about the code as written.all_or_nothingis the second, for a batch whose parts are one edit rather than several. A veto normally skips the proposal it lands on and lets the rest through, which is right when each stands alone; it is wrong when a partly applied batch is not a smaller edit but a broken one.Ps1ControlFlowDeflatteningreplaces a dispatcher loop and deletes the$state = ...seeding it, and applying either alone leaves a state machine half dissolved. Neither the class nor the veto can tell the two cases apart, so the pass says which it is.Expand source code Browse git
class Ps1RemovalPlan: """ A batch of proposed edits to one statement list, committed as a single mutation. A pass proposes each edit it wants, consults the set-level guards it is responsible for against `survivors`, and calls `commit`. What the class owns is the part every pass has to get right the same way: the per-statement veto, and the fact that the whole batch lands as one tree edit. **Which set a guard must be shown is decided by its polarity, and the two answers are opposite.** A guard *permissive* in the survivor set — more survivors means a removal is more likely to be allowed — must see the **pre-veto** set, which is `survivors`. Both `refinery.lib.scripts.ps1.analysis.effects.output_is_covered` and `refinery.lib.scripts.ps1.analysis.effects.pruning_erases_body` are this kind, so a guard shown the post-veto set would read a vetoed statement as cover for deleting its neighbour, and a statement kept because a handler observes its fault would license destroying the one beside it. The veto therefore runs inside `commit`, after every such guard has had its answer, and `survivors` cannot show one a veto. A guard *restrictive* in it — more survivors means a removal is **less** likely to be allowed — must see the post-veto set, or a vetoed statement's dependencies are deleted out from under it. Reachability and liveness are this kind, and they read `accepted`, which reports what this plan would edit rather than reconstituting a survivor set the permissive guards could reach. Set-level guards deliberately stay with the passes. They are not the same from one pass to the next — `Ps1DeadCodeElimination` declines `output_is_covered` as too permissive for what it does, and `Ps1JunkStatementRemoval._remove_inert_functions` weighs erasure against the definitions alone rather than against everything it removes — so collapsing them into one call here would have to be spelled as flags, which is the policy sprawl this class exists to reduce. `removals_may_fault` is the one thing about a pass this class does have to be told, because the fault refusal is not the same question for every pass either. `refinery.lib.scripts.ps1.analysis.effects.fault_is_observed` exists because *nothing* in the effect substrate can answer whether a statement throws, so a pass that cannot say declines every removal from a protected `try` body. `Ps1DeadCodeElimination` can say: it removes pure constants and constructs whose condition it has already proved constant, neither of which can raise. For such a pass the refusal that remains is the set-level one the per-statement veto was standing in for — the batch must not leave the protected body empty, because an empty `try` block is evidence about the pass rather than about the code as written. `all_or_nothing` is the second, for a batch whose parts are one edit rather than several. A veto normally skips the proposal it lands on and lets the rest through, which is right when each stands alone; it is wrong when a partly applied batch is not a smaller edit but a broken one. `Ps1ControlFlowDeflattening` replaces a dispatcher loop and deletes the `$state = ...` seeding it, and applying either alone leaves a state machine half dissolved. Neither the class nor the veto can tell the two cases apart, so the pass says which it is. """ def __init__( self, parent: Node, attr: str = 'body', removals_may_fault: bool = True, all_or_nothing: bool = False, ): self.parent = parent self.attr = attr self.removals_may_fault = removals_may_fault self.all_or_nothing = all_or_nothing self._proposals: dict[int, _Proposal] = {} def propose( self, statement: Statement, replacement: list[Statement] | None = None, ) -> None: """ Register that `statement` is to be replaced by `replacement`, or removed when that is `None` or empty. Proposing the same statement twice keeps the later proposal. A replacement is not a weaker removal: `Ps1DeadCodeElimination` resolves a constant `if` into the statements of the branch that runs, and a dead store becomes `$Null = <rhs>` so the value is still computed. A pass that could only delete could express neither. **A registered replacement holds no claim on the tree until `commit` grants it one.** Building one adopts the parts of the original it reuses, and the original is still standing, so the adoption leaves nodes in the tree naming a holder that is not; the statement is put back in order here, before this call returns. Everything that happens between a proposal and the verdict reads the tree by walking upward — the set-level guards, the veto, the search for the list a statement sits in — and a batch that decides against a tree it has already half detached decides about a tree that does not exist. Making the repair a condition of registering is also what lets a pass build every replacement up front and withdraw or abandon afterwards without owing anything. The repair is owed by every registration, not only by one that ends up carrying a replacement. What has to be given back is what the *caller* built, and a pass routinely builds a replacement and then decides against installing it before it ever gets here: `Ps1DeadCodeElimination` hoists a construct's branch into new statements and only afterwards drops the ones its set-level guard forbids, which can empty the list. Reading the argument that arrives as the record of what was built is reading the survivors of that filter, and it left a statement standing over a literal that named a node the pass had thrown away. """ self._proposals[id(statement)] = _Proposal(statement, list(replacement or ())) reattach(statement) def withdraw(self, statement: Statement) -> None: """ Drop a registered proposal, leaving `statement` where it stands. A pass that shrinks its own batch after reading `accepted` uses this. Nothing needs putting back, because `propose` never let the replacement take anything in the first place. """ self._proposals.pop(id(statement), None) def abandon(self) -> None: """ Drop every proposal, leaving the tree as it was. Same contract as `withdraw`, for a pass that built a whole batch and then decided against all of it. """ self._proposals.clear() @property def survivors(self) -> list[Statement]: """ The list as it would stand if every proposal were applied, the veto ignored. This is what the set-level guards must be shown; see the class docstring for why the post-veto set must not reach them. """ return self._edit(self._proposals.values()).result() @property def accepted(self) -> list[Statement]: """ The statements `commit` would edit, without editing them. This is for the *restrictive* guards — the ones that allow **fewer** removals as more statements survive, which is the opposite polarity to `survivors`' readers. Reachability is the example: it concludes a function is dead from the call sites that are going away, so a vetoed caller it never heard about leaves the emitted script calling a function it does not define. Such a guard asks this, drops what it now forbids with `withdraw`, and asks again; the loop terminates because the batch only shrinks. **That last part is a fact about the default flags, not about this query.** Under `all_or_nothing`, and under `removals_may_fault=False` against a protected body, a withdrawal can take this set from empty to non-empty — the batch *grows*, and a loop resting on the shrinking argument does not terminate. `Ps1RemovalPlans` opens every plan with the defaults, which is why its consumers may write that loop; a pass that sets either flag and then loops on `accepted` owes its own termination argument. What a caller obtains is what *this* plan would do, not a post-veto survivor set. The distinction is the whole safety argument: `survivors` still cannot show a permissive guard a veto, so a statement the veto keeps never becomes licence to delete the one beside it. This must not edit the tree. A query that installs a replacement's claim on its children is a query that decides the batch, and the guard asking it has not decided anything yet; the claims are granted in `commit`, where the verdict is final. A caller may read this as exact, and every consumer does: membership means *this is going away*, so a statement reported here that `commit` then leaves standing is a rescue that never happens and a dependency deleted out from under it — the failure the restrictive polarity exists to prevent, not the safe side of it. The verdict a plan gives here is the one it applies, and `Ps1RemovalPlans` reaches every verdict before it lands the first edit so that stays true across a batch. What that exactness rests on is the plan's list actually holding what was proposed against it, which is a fact about the proposal and not one this query can establish: `BodyEdit` ignores a splice for a node its list does not hold, and `_apply` reports such a proposal as landing nothing. `Ps1RemovalPlans.propose` establishes it by finding the list, and `Ps1RemovalPlans.propose_in` moves it to the caller, which is what a caller of that method takes on. """ return [proposal.statement for proposal in self._allowed()] def _edit(self, proposals) -> BodyEdit: edit = BodyEdit(self.parent, self.attr) for proposal in proposals: edit.splice(proposal.statement, proposal.replacement) return edit def _vetoed(self, proposal: _Proposal) -> bool: """ Whether a single proposal must be skipped although the guards allowed the batch. `fault_is_observed` asks whether the edit changes which code runs, and it is asked of deletions only, because a rewrite that keeps evaluating the original expression still throws where the original threw, so an enclosing handler stays as reachable as it was. It is asked only of a pass that cannot rule the fault out itself — or of a statement that *is* a handler, which no pass can rule out; see `_removes_a_handler`. """ if proposal.replacement: return False if not (self.removals_may_fault or _removes_a_handler(proposal.statement)): return False return fault_is_observed(proposal.statement) def _allowed(self) -> list[_Proposal]: """ The proposals that survive the veto and every set-level refusal this class owns. Kept apart from `commit` so that `accepted` is a query over the decision rather than a second copy of it. """ proposals = list(self._proposals.values()) allowed = [p for p in proposals if not self._vetoed(p)] if self.all_or_nothing and len(allowed) != len(proposals): allowed = [] if self._empties_a_protected_body(allowed): allowed = [] return allowed def commit(self) -> bool: """ Apply every proposal no veto blocks, as one edit, and report whether the tree moved. """ moved, landed = self._apply(self._allowed()) _restore(landed) return moved def _apply(self, allowed: list[_Proposal]) -> tuple[bool, list[Statement]]: """ Land one already-reached verdict, reporting whether the tree moved and which nodes the list now holds because of it. Split out for `Ps1RemovalPlans`, which has to reach every verdict before it lands any of them, and land every edit before it repairs any of them. What landed is decided per splice rather than taken from `allowed`, because the two are not the same set. `BodyEdit` ignores a splice for a node its list does not hold — the class describes an edit to one list and nothing else — so a replacement can be allowed and still never be installed. Repairing that one would hand it the children it is still not holding, which is the corruption `propose` undoes at registration, reintroduced by the repair. The question a splice was honoured is asked of the statement it names and not of the resulting list, and only the first is the same question: a replacement that already stands in the list is carried over by an edit that ignored its splice, so reading the result back reports it installed by an edit that installed nothing. """ if not allowed: return False, [] held = {id(item) for item in getattr(self.parent, self.attr, None) or []} if not self._edit(allowed).apply(): return False, [] return True, [ statement for proposal in allowed if id(proposal.statement) in held for statement in proposal.replacement ] def _empties_a_protected_body(self, allowed: list[_Proposal]) -> bool: """ Whether committing `allowed` would clear a `try` body beside a handler that acts. Asked only of a pass that ruled the per-statement fault question out; for every other pass the veto has already declined each such removal one at a time. The emptiness test lives here rather than at the call site so that the two halves of the name are decided in one place, and so that the `BodyEdit` it needs is built only for the pass that can use the answer. """ if self.removals_may_fault or not allowed: return False if self._edit(allowed).result(): return False return any(fault_is_observed(proposal.statement) for proposal in allowed)Instance variables
var survivors-
The list as it would stand if every proposal were applied, the veto ignored. This is what the set-level guards must be shown; see the class docstring for why the post-veto set must not reach them.
Expand source code Browse git
@property def survivors(self) -> list[Statement]: """ The list as it would stand if every proposal were applied, the veto ignored. This is what the set-level guards must be shown; see the class docstring for why the post-veto set must not reach them. """ return self._edit(self._proposals.values()).result() var accepted-
The statements
commitwould edit, without editing them.This is for the restrictive guards — the ones that allow fewer removals as more statements survive, which is the opposite polarity to
survivors' readers. Reachability is the example: it concludes a function is dead from the call sites that are going away, so a vetoed caller it never heard about leaves the emitted script calling a function it does not define. Such a guard asks this, drops what it now forbids withwithdraw, and asks again; the loop terminates because the batch only shrinks.That last part is a fact about the default flags, not about this query. Under
all_or_nothing, and underremovals_may_fault=Falseagainst a protected body, a withdrawal can take this set from empty to non-empty — the batch grows, and a loop resting on the shrinking argument does not terminate.Ps1RemovalPlansopens every plan with the defaults, which is why its consumers may write that loop; a pass that sets either flag and then loops onacceptedowes its own termination argument.What a caller obtains is what this plan would do, not a post-veto survivor set. The distinction is the whole safety argument:
survivorsstill cannot show a permissive guard a veto, so a statement the veto keeps never becomes licence to delete the one beside it.This must not edit the tree. A query that installs a replacement's claim on its children is a query that decides the batch, and the guard asking it has not decided anything yet; the claims are granted in
commit, where the verdict is final.A caller may read this as exact, and every consumer does: membership means this is going away, so a statement reported here that
committhen leaves standing is a rescue that never happens and a dependency deleted out from under it — the failure the restrictive polarity exists to prevent, not the safe side of it. The verdict a plan gives here is the one it applies, andPs1RemovalPlansreaches every verdict before it lands the first edit so that stays true across a batch.What that exactness rests on is the plan's list actually holding what was proposed against it, which is a fact about the proposal and not one this query can establish:
BodyEditignores a splice for a node its list does not hold, and_applyreports such a proposal as landing nothing.Ps1RemovalPlans.propose()establishes it by finding the list, andPs1RemovalPlans.propose_in()moves it to the caller, which is what a caller of that method takes on.Expand source code Browse git
@property def accepted(self) -> list[Statement]: """ The statements `commit` would edit, without editing them. This is for the *restrictive* guards — the ones that allow **fewer** removals as more statements survive, which is the opposite polarity to `survivors`' readers. Reachability is the example: it concludes a function is dead from the call sites that are going away, so a vetoed caller it never heard about leaves the emitted script calling a function it does not define. Such a guard asks this, drops what it now forbids with `withdraw`, and asks again; the loop terminates because the batch only shrinks. **That last part is a fact about the default flags, not about this query.** Under `all_or_nothing`, and under `removals_may_fault=False` against a protected body, a withdrawal can take this set from empty to non-empty — the batch *grows*, and a loop resting on the shrinking argument does not terminate. `Ps1RemovalPlans` opens every plan with the defaults, which is why its consumers may write that loop; a pass that sets either flag and then loops on `accepted` owes its own termination argument. What a caller obtains is what *this* plan would do, not a post-veto survivor set. The distinction is the whole safety argument: `survivors` still cannot show a permissive guard a veto, so a statement the veto keeps never becomes licence to delete the one beside it. This must not edit the tree. A query that installs a replacement's claim on its children is a query that decides the batch, and the guard asking it has not decided anything yet; the claims are granted in `commit`, where the verdict is final. A caller may read this as exact, and every consumer does: membership means *this is going away*, so a statement reported here that `commit` then leaves standing is a rescue that never happens and a dependency deleted out from under it — the failure the restrictive polarity exists to prevent, not the safe side of it. The verdict a plan gives here is the one it applies, and `Ps1RemovalPlans` reaches every verdict before it lands the first edit so that stays true across a batch. What that exactness rests on is the plan's list actually holding what was proposed against it, which is a fact about the proposal and not one this query can establish: `BodyEdit` ignores a splice for a node its list does not hold, and `_apply` reports such a proposal as landing nothing. `Ps1RemovalPlans.propose` establishes it by finding the list, and `Ps1RemovalPlans.propose_in` moves it to the caller, which is what a caller of that method takes on. """ return [proposal.statement for proposal in self._allowed()]
Methods
def propose(self, statement, replacement=None)-
Register that
statementis to be replaced byreplacement, or removed when that isNoneor empty. Proposing the same statement twice keeps the later proposal.A replacement is not a weaker removal:
Ps1DeadCodeEliminationresolves a constantifinto the statements of the branch that runs, and a dead store becomes$Null = <rhs>so the value is still computed. A pass that could only delete could express neither.A registered replacement holds no claim on the tree until
commitgrants it one. Building one adopts the parts of the original it reuses, and the original is still standing, so the adoption leaves nodes in the tree naming a holder that is not; the statement is put back in order here, before this call returns. Everything that happens between a proposal and the verdict reads the tree by walking upward — the set-level guards, the veto, the search for the list a statement sits in — and a batch that decides against a tree it has already half detached decides about a tree that does not exist. Making the repair a condition of registering is also what lets a pass build every replacement up front and withdraw or abandon afterwards without owing anything.The repair is owed by every registration, not only by one that ends up carrying a replacement. What has to be given back is what the caller built, and a pass routinely builds a replacement and then decides against installing it before it ever gets here:
Ps1DeadCodeEliminationhoists a construct's branch into new statements and only afterwards drops the ones its set-level guard forbids, which can empty the list. Reading the argument that arrives as the record of what was built is reading the survivors of that filter, and it left a statement standing over a literal that named a node the pass had thrown away.Expand source code Browse git
def propose( self, statement: Statement, replacement: list[Statement] | None = None, ) -> None: """ Register that `statement` is to be replaced by `replacement`, or removed when that is `None` or empty. Proposing the same statement twice keeps the later proposal. A replacement is not a weaker removal: `Ps1DeadCodeElimination` resolves a constant `if` into the statements of the branch that runs, and a dead store becomes `$Null = <rhs>` so the value is still computed. A pass that could only delete could express neither. **A registered replacement holds no claim on the tree until `commit` grants it one.** Building one adopts the parts of the original it reuses, and the original is still standing, so the adoption leaves nodes in the tree naming a holder that is not; the statement is put back in order here, before this call returns. Everything that happens between a proposal and the verdict reads the tree by walking upward — the set-level guards, the veto, the search for the list a statement sits in — and a batch that decides against a tree it has already half detached decides about a tree that does not exist. Making the repair a condition of registering is also what lets a pass build every replacement up front and withdraw or abandon afterwards without owing anything. The repair is owed by every registration, not only by one that ends up carrying a replacement. What has to be given back is what the *caller* built, and a pass routinely builds a replacement and then decides against installing it before it ever gets here: `Ps1DeadCodeElimination` hoists a construct's branch into new statements and only afterwards drops the ones its set-level guard forbids, which can empty the list. Reading the argument that arrives as the record of what was built is reading the survivors of that filter, and it left a statement standing over a literal that named a node the pass had thrown away. """ self._proposals[id(statement)] = _Proposal(statement, list(replacement or ())) reattach(statement) def withdraw(self, statement)-
Drop a registered proposal, leaving
statementwhere it stands. A pass that shrinks its own batch after readingaccepteduses this. Nothing needs putting back, becauseproposenever let the replacement take anything in the first place.Expand source code Browse git
def withdraw(self, statement: Statement) -> None: """ Drop a registered proposal, leaving `statement` where it stands. A pass that shrinks its own batch after reading `accepted` uses this. Nothing needs putting back, because `propose` never let the replacement take anything in the first place. """ self._proposals.pop(id(statement), None) def abandon(self)-
Drop every proposal, leaving the tree as it was. Same contract as
withdraw, for a pass that built a whole batch and then decided against all of it.Expand source code Browse git
def abandon(self) -> None: """ Drop every proposal, leaving the tree as it was. Same contract as `withdraw`, for a pass that built a whole batch and then decided against all of it. """ self._proposals.clear() def commit(self)-
Apply every proposal no veto blocks, as one edit, and report whether the tree moved.
Expand source code Browse git
def commit(self) -> bool: """ Apply every proposal no veto blocks, as one edit, and report whether the tree moved. """ moved, landed = self._apply(self._allowed()) _restore(landed) return moved
class Ps1RemovalPlans-
One
Ps1RemovalPlanper statement list, for a pass that finds its removals by walking the whole tree rather than by descending body by body. Each list still commits as a single edit, so a pass scattering removals across a script advances the mutation counter once per body it touches instead of once per statement.A whole-tree walk also reaches statements that sit in no list at all: the inner store of
($y = ($z = 1))is a statement to every pass that finds it and a direct field to its parent. Those are carried here too, because a pass that finds one has no other route left — but only as rewrites. A field cannot lose its statement without the parent losing its shape, so a proposal to remove one outright is registered and then declined at commit, which is also why the fault veto has nothing to say about them: it declines deletions, and none of these is one.Expand source code Browse git
class Ps1RemovalPlans: """ One `Ps1RemovalPlan` per statement list, for a pass that finds its removals by walking the whole tree rather than by descending body by body. Each list still commits as a single edit, so a pass scattering removals across a script advances the mutation counter once per body it touches instead of once per statement. A whole-tree walk also reaches statements that sit in no list at all: the inner store of `($y = ($z = 1))` is a statement to every pass that finds it and a direct field to its parent. Those are carried here too, because a pass that finds one has no other route left — but only as rewrites. A field cannot lose its statement without the parent losing its shape, so a proposal to remove one outright is registered and then declined at commit, which is also why the fault veto has nothing to say about them: it declines deletions, and none of these is one. """ def __init__(self): self._plans: dict[tuple[int, str], Ps1RemovalPlan] = {} self._rewrites: dict[int, _Proposal] = {} #: The filed statement is kept beside its plan, and not only its `id`, for the reason #: `refinery.lib.scripts.BodyEdit` keeps a spliced node beside its own: a statement that is #: collected while its entry stands would let the next object at that address be withdrawn #: from a plan that never held it. self._filed: dict[int, tuple[Statement, Ps1RemovalPlan]] = {} def propose_in( self, parent: Node, statement: Statement, replacement: list[Statement] | None = None, attr: str = 'body', ) -> None: """ Register an edit against the list `parent.<attr>`, which the caller states holds `statement`. `propose` has to find that list, and finding it is an identity scan over the list — the cost of one proposal is the length of the body, so the cost of a pass is the square of it. A pass that walks bodies to find its removals is already holding the list, and says so here. Nothing checks the claim, so a caller that names the wrong list files a proposal `commit` will silently drop and `accepted` will still report; see `Ps1RemovalPlan.accepted` for what rests on it. Filing the same statement a second time drops the first proposal rather than leaving it standing, because the alternative is a statement `withdraw` can only reach one of — a withdrawal that half happens is what remembering where a proposal landed exists to rule out. """ plan = self._plan_for(parent, attr) filed = self._filed.get(id(statement)) if filed is not None and filed[1] is not plan: filed[1].withdraw(statement) plan.propose(statement, replacement) self._filed[id(statement)] = (statement, plan) def _plan_for(self, parent: Node, attr: str) -> Ps1RemovalPlan: key = (id(parent), attr) try: return self._plans[key] except KeyError: plan = self._plans[key] = Ps1RemovalPlan(parent, attr) return plan def propose( self, statement: Statement, replacement: list[Statement] | None = None, ) -> bool: """ Register an edit with the plan for the list holding `statement`, opening one if this is the first edit against that list, or as a direct-field rewrite when `statement` sits in no list. Reports whether the statement can be edited at all. A refusal releases the proposal rather than handing it back: the caller has already built its replacement, and building one adopts parts of the statement, so a refusal the caller has to remember to undo is a refusal that gets forgotten. See `Ps1RemovalPlan.propose` for why no registered replacement holds a claim before `commit` either, and for why the release is owed whatever the `replacement` argument turns out to hold. """ owner = owning_list(statement) if owner is None: if owning_field(statement) is None: reattach(statement) return False self._rewrites[id(statement)] = _Proposal(statement, list(replacement or ())) reattach(statement) return True parent, attr = owner self.propose_in(parent, statement, replacement, attr) return True def withdraw(self, statement: Statement) -> None: """ Drop a registered proposal wherever it landed. Same contract as `Ps1RemovalPlan.withdraw`. Where it landed is remembered rather than looked up again. Rediscovering the owning list reports nothing when it fails, and a withdrawal that quietly does not happen is a proposal the caller has already written off and `commit` still applies — half of a group edit whose other half is gone. """ if self._rewrites.pop(id(statement), None) is not None: return filed = self._filed.pop(id(statement), None) if filed is not None: filed[1].withdraw(statement) def abandon(self) -> None: """ Drop every proposal in every plan. Same contract as `Ps1RemovalPlan.abandon`. """ for plan in self._plans.values(): plan.abandon() self._rewrites.clear() self._filed.clear() def survivors(self, parent: Node, attr: str = 'body') -> list[Statement]: """ The pre-veto survivors of one of the lists this batch touches, or its current contents when no edit was registered against it. Same contract as `Ps1RemovalPlan.survivors`. """ plan = self._plans.get((id(parent), attr)) if plan is None: return list(getattr(parent, attr, None) or []) return plan.survivors @property def accepted(self) -> list[Statement]: """ The statements `commit` would edit across every list this batch touches, plus the direct-field rewrites it would install. Same contract, and the same two limits, as `Ps1RemovalPlan.accepted`. A pass that scatters one logical removal across several lists needs this rather than the per-plan answer: `refinery.lib.scripts.ps1.deobfuscation.unused.Ps1JunkStatementRemoval` drops an inert definition and the bare calls to it, and those routinely land in different plans, so a veto on either half is only visible here. """ accepted = [statement for plan in self._plans.values() for statement in plan.accepted] accepted.extend(proposal.statement for proposal, _ in self._installable()) return accepted def _installable(self) -> list[tuple[_Proposal, Node]]: """ The direct-field rewrites this batch would install, each beside the node the field takes. One decision, read by `accepted` and applied by `commit`, so the two cannot drift. """ installable = [] for proposal in self._rewrites.values(): replacement = _fits_a_field(proposal) if replacement is None: continue installable.append((proposal, replacement)) return installable def commit(self) -> bool: """ Commit every plan and report whether any of them moved the tree. Every verdict is reached before the first edit lands, and every edit lands before the first repair. A veto is a question about the tree — `refinery.lib.scripts.ps1.analysis.effects.fault_is_observed` reads the handler beside the body it is asked about — so a plan that emptied a `catch` body would change the answer for the `try` body's plan, and which plan that is would be decided by nothing better than the order the batch happened to open them in. That is also what makes `accepted` exact: what it reported is what commits. `_restore` says why the repairs come last. Nothing that did not land is repaired. A rewrite the field refused is a replacement that was released when it was registered and has taken nothing since, so its original owes nothing either — and asserting an uninstalled statement's structure here is the one walk that could assert it over a node the tree holds somewhere else. """ verdicts = [(plan, plan._allowed()) for plan in self._plans.values()] rewrites = self._installable() landed: list[Node] = [] moved = False for plan, allowed in verdicts: was_moved, installed = plan._apply(allowed) moved = moved or was_moved landed.extend(installed) for proposal, replacement in rewrites: if not _replace_in_parent(proposal.statement, replacement): continue landed.append(replacement) moved = True _restore(landed) return movedInstance variables
var accepted-
The statements
commitwould edit across every list this batch touches, plus the direct-field rewrites it would install. Same contract, and the same two limits, asPs1RemovalPlan.accepted.A pass that scatters one logical removal across several lists needs this rather than the per-plan answer:
Ps1JunkStatementRemovaldrops an inert definition and the bare calls to it, and those routinely land in different plans, so a veto on either half is only visible here.Expand source code Browse git
@property def accepted(self) -> list[Statement]: """ The statements `commit` would edit across every list this batch touches, plus the direct-field rewrites it would install. Same contract, and the same two limits, as `Ps1RemovalPlan.accepted`. A pass that scatters one logical removal across several lists needs this rather than the per-plan answer: `refinery.lib.scripts.ps1.deobfuscation.unused.Ps1JunkStatementRemoval` drops an inert definition and the bare calls to it, and those routinely land in different plans, so a veto on either half is only visible here. """ accepted = [statement for plan in self._plans.values() for statement in plan.accepted] accepted.extend(proposal.statement for proposal, _ in self._installable()) return accepted
Methods
def propose_in(self, parent, statement, replacement=None, attr='body')-
Register an edit against the list
parent.<attr>, which the caller states holdsstatement.proposehas to find that list, and finding it is an identity scan over the list — the cost of one proposal is the length of the body, so the cost of a pass is the square of it. A pass that walks bodies to find its removals is already holding the list, and says so here.Nothing checks the claim, so a caller that names the wrong list files a proposal
commitwill silently drop andacceptedwill still report; seePs1RemovalPlan.acceptedfor what rests on it. Filing the same statement a second time drops the first proposal rather than leaving it standing, because the alternative is a statementwithdrawcan only reach one of — a withdrawal that half happens is what remembering where a proposal landed exists to rule out.Expand source code Browse git
def propose_in( self, parent: Node, statement: Statement, replacement: list[Statement] | None = None, attr: str = 'body', ) -> None: """ Register an edit against the list `parent.<attr>`, which the caller states holds `statement`. `propose` has to find that list, and finding it is an identity scan over the list — the cost of one proposal is the length of the body, so the cost of a pass is the square of it. A pass that walks bodies to find its removals is already holding the list, and says so here. Nothing checks the claim, so a caller that names the wrong list files a proposal `commit` will silently drop and `accepted` will still report; see `Ps1RemovalPlan.accepted` for what rests on it. Filing the same statement a second time drops the first proposal rather than leaving it standing, because the alternative is a statement `withdraw` can only reach one of — a withdrawal that half happens is what remembering where a proposal landed exists to rule out. """ plan = self._plan_for(parent, attr) filed = self._filed.get(id(statement)) if filed is not None and filed[1] is not plan: filed[1].withdraw(statement) plan.propose(statement, replacement) self._filed[id(statement)] = (statement, plan) def propose(self, statement, replacement=None)-
Register an edit with the plan for the list holding
statement, opening one if this is the first edit against that list, or as a direct-field rewrite whenstatementsits in no list. Reports whether the statement can be edited at all.A refusal releases the proposal rather than handing it back: the caller has already built its replacement, and building one adopts parts of the statement, so a refusal the caller has to remember to undo is a refusal that gets forgotten. See
Ps1RemovalPlan.propose()for why no registered replacement holds a claim beforecommiteither, and for why the release is owed whatever thereplacementargument turns out to hold.Expand source code Browse git
def propose( self, statement: Statement, replacement: list[Statement] | None = None, ) -> bool: """ Register an edit with the plan for the list holding `statement`, opening one if this is the first edit against that list, or as a direct-field rewrite when `statement` sits in no list. Reports whether the statement can be edited at all. A refusal releases the proposal rather than handing it back: the caller has already built its replacement, and building one adopts parts of the statement, so a refusal the caller has to remember to undo is a refusal that gets forgotten. See `Ps1RemovalPlan.propose` for why no registered replacement holds a claim before `commit` either, and for why the release is owed whatever the `replacement` argument turns out to hold. """ owner = owning_list(statement) if owner is None: if owning_field(statement) is None: reattach(statement) return False self._rewrites[id(statement)] = _Proposal(statement, list(replacement or ())) reattach(statement) return True parent, attr = owner self.propose_in(parent, statement, replacement, attr) return True def withdraw(self, statement)-
Drop a registered proposal wherever it landed. Same contract as
Ps1RemovalPlan.withdraw().Where it landed is remembered rather than looked up again. Rediscovering the owning list reports nothing when it fails, and a withdrawal that quietly does not happen is a proposal the caller has already written off and
commitstill applies — half of a group edit whose other half is gone.Expand source code Browse git
def withdraw(self, statement: Statement) -> None: """ Drop a registered proposal wherever it landed. Same contract as `Ps1RemovalPlan.withdraw`. Where it landed is remembered rather than looked up again. Rediscovering the owning list reports nothing when it fails, and a withdrawal that quietly does not happen is a proposal the caller has already written off and `commit` still applies — half of a group edit whose other half is gone. """ if self._rewrites.pop(id(statement), None) is not None: return filed = self._filed.pop(id(statement), None) if filed is not None: filed[1].withdraw(statement) def abandon(self)-
Drop every proposal in every plan. Same contract as
Ps1RemovalPlan.abandon().Expand source code Browse git
def abandon(self) -> None: """ Drop every proposal in every plan. Same contract as `Ps1RemovalPlan.abandon`. """ for plan in self._plans.values(): plan.abandon() self._rewrites.clear() self._filed.clear() def survivors(self, parent, attr='body')-
The pre-veto survivors of one of the lists this batch touches, or its current contents when no edit was registered against it. Same contract as
Ps1RemovalPlan.survivors.Expand source code Browse git
def survivors(self, parent: Node, attr: str = 'body') -> list[Statement]: """ The pre-veto survivors of one of the lists this batch touches, or its current contents when no edit was registered against it. Same contract as `Ps1RemovalPlan.survivors`. """ plan = self._plans.get((id(parent), attr)) if plan is None: return list(getattr(parent, attr, None) or []) return plan.survivors def commit(self)-
Commit every plan and report whether any of them moved the tree.
Every verdict is reached before the first edit lands, and every edit lands before the first repair. A veto is a question about the tree —
fault_is_observed()reads the handler beside the body it is asked about — so a plan that emptied acatchbody would change the answer for thetrybody's plan, and which plan that is would be decided by nothing better than the order the batch happened to open them in. That is also what makesacceptedexact: what it reported is what commits._restoresays why the repairs come last.Nothing that did not land is repaired. A rewrite the field refused is a replacement that was released when it was registered and has taken nothing since, so its original owes nothing either — and asserting an uninstalled statement's structure here is the one walk that could assert it over a node the tree holds somewhere else.
Expand source code Browse git
def commit(self) -> bool: """ Commit every plan and report whether any of them moved the tree. Every verdict is reached before the first edit lands, and every edit lands before the first repair. A veto is a question about the tree — `refinery.lib.scripts.ps1.analysis.effects.fault_is_observed` reads the handler beside the body it is asked about — so a plan that emptied a `catch` body would change the answer for the `try` body's plan, and which plan that is would be decided by nothing better than the order the batch happened to open them in. That is also what makes `accepted` exact: what it reported is what commits. `_restore` says why the repairs come last. Nothing that did not land is repaired. A rewrite the field refused is a replacement that was released when it was registered and has taken nothing since, so its original owes nothing either — and asserting an uninstalled statement's structure here is the one walk that could assert it over a node the tree holds somewhere else. """ verdicts = [(plan, plan._allowed()) for plan in self._plans.values()] rewrites = self._installable() landed: list[Node] = [] moved = False for plan, allowed in verdicts: was_moved, installed = plan._apply(allowed) moved = moved or was_moved landed.extend(installed) for proposal, replacement in rewrites: if not _replace_in_parent(proposal.statement, replacement): continue landed.append(replacement) moved = True _restore(landed) return moved