fix(agent-client): only fire change hook for fields that declare one

[nbouliol]

Problem

Running a smart action through the workflow executor (trigger-record-action, AI-assisted or Full AI) crashes against Ruby (forest_liana) agents: getActionForm failed: Agent responded with HTTP 500, the agent raising NoMethodError: undefined method 'call' for nil:NilClass. Manual form fill in the Ember UI works; Node agents tolerate it.

Root cause

FieldFormStates.setFieldValue posted to /hooks/change for any field as soon as the action declared at least one change hook, without checking the changed field had its own hook. tryToSetFields iterates every AI-filled field, so setting a hookless field (e.g. code) sent changed_field for it; forest_liana then evaluated action.hooks[:change][nil].call and raised. The Ember UI only fires a change for fields that declare a hook.

Fix

• Gate loadChanges on the changed field's own hook, matching the admin UI.
• Thread the per-field hook through PlainField, ForestSchemaAction.fields, and the loadInitialState fallback mapping so it is reliably available (Node sets changeHook only on watched fields; Ruby sets the hook name).

Tests

field-form-states suite green (20/20), incl. a new regression test: a hookless field does not trigger a change even when the action has change hooks.

Fixes PRD-778

🤖 Generated with Claude Code

Note

Fix setFieldValue to only fire change hook for fields that declare one

Previously, FieldFormStates.setFieldValue triggered a POST to /hooks/change based on whether hooks.change had entries, regardless of whether the specific changed field declared a hook. Now it checks field.getPlainField().hook and skips the request for fields without a hook.

• Adds an optional hook?: string property to PlainField in types.ts and ForestSchemaAction fields in forestadmin-client types.ts.
• Updates loadInitialState to forward the hook property when mapping fallback fields, so hook-based change behavior works correctly for those fields too.
• Behavioral Change: fields without a declared hook no longer trigger change hook requests even if the action has hooks.change defined.

^{Macroscope summarized a243be3.}

[linear-code]

PRD-778

[qltysh]

Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (1)

Rating	File	% Diff	Uncovered Line #s
	packages/agent-client/src/action-fields/field-form-states.ts	100.0%
	Total	100.0%

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.