You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The JSON actions[] array has two semantically distinct shapes of auto_fixable:
Per-rule: the rule type can be auto-fixed (most actions today). Constant across every finding for that issue type. Example: every remove-export action has auto_fixable: true.
The distinction matters for agents that filter on auto_fixable: true to decide what's safe to apply blindly. Per-instance semantics is the correct model (matches ESLint's RuleFix | RuleFix[] | null return shape), but it's currently undocumented.
Proposed solution
Add a _meta entry in crates/cli/src/explain.rs documenting auto_fixable as per-instance ("evaluated per finding") and citing the issue types that currently use per-instance flipping.
Add the same note to docs/output-schema.json next to the actions[].auto_fixable field definition.
Update fallow-docs/analysis/auto-fix.mdx and fallow-skills cli-reference.md with the same wording.
Follow-up to #335.
The JSON
actions[]array has two semantically distinct shapes ofauto_fixable:remove-exportaction hasauto_fixable: true.remove-catalog-entryisauto_fixable: truewhenhardcoded_consumersis empty,falseotherwise. Sibling:update-catalog-referencefrom Unresolved catalog reference: warn when consumer uses catalog:X but X catalog lacks the package #334 isauto_fixable: falseuntilavailable_in_catalogshas exactly one entry.The distinction matters for agents that filter on
auto_fixable: trueto decide what's safe to apply blindly. Per-instance semantics is the correct model (matches ESLint'sRuleFix | RuleFix[] | nullreturn shape), but it's currently undocumented.Proposed solution
_metaentry incrates/cli/src/explain.rsdocumentingauto_fixableas per-instance ("evaluated per finding") and citing the issue types that currently use per-instance flipping.docs/output-schema.jsonnext to theactions[].auto_fixablefield definition.fallow-docs/analysis/auto-fix.mdxandfallow-skillscli-reference.md with the same wording.Low-priority polish; no behavior change.