docs: cleanup module and state docstrings to match behavior (master)

[dwoz]

Summary

Batch cleanup of state and execution-module docstrings to match
actual behavior. No code changes; documentation only.

Issues addressed (each linked below as Refs #N)

• Inconsistent argument naming in salt.modules.slack_notify.call_hook #58845 slack_notify.call_hook — documented the configuration key
  as identifier instead of hook; added a minion-config example.
• [DOCS] salt.modules.file.seek_read contains incorrect arg name in the description #67074 salt.modules.file.seek_read — second parameter is size,
  not seek.
• [DOCS] option "user" is misnamed. It should be "owner" #67911 salt.modules.file.find — the option name is owner, not
  user.
• Inconsistent documentation of "enabled" parameter in states.pkgrepo #54802 salt.states.pkgrepo.managed — passing enabled=False is
  equivalent to disabled=True, not disabled=False.
• [DOCS] Please add note about trying the "hkp://" syntax for pkgrepo.managed #61671 salt.states.pkgrepo.managed — added the hkp:// keyserver
  workaround note.
• Missing API documentation for __func_alias__es #62002 salt.wheel.key — documented the __func_alias__ mapping
  (list_ → key.list, key_str → key.print).
• [BUG] Documentation bug on state module virtualenv #56729 / [DOCS] virtualenv state module option #65756 salt.states.virtualenv_mod — surfaced the
  virtualenv virtual name and pointed at
  salt.modules.virtualenv_mod.create for unmapped kwargs.
• [DOCS] aptpkg.held state does not exist #61886 salt.states.aptpkg — surfaced the apt virtual name.
• Error in salt module documentation #59666 salt.modules.groupadd — surfaced the group virtual name.
• [documentation] Documentation for states.file.rename should be updated to mention that it works for directories too. #55916 salt.states.file.rename — documented directory support.
• [doc] salt.modules.file copy addition #50568 salt.states.file.copy_ — added a pointer to
  salt.modules.cp.get_file.
• [DOCS] In file.blockreplace "marker_end" cannot contain "marker_start" #64075 salt.states.file.blockreplace — documented the
  start/end marker overlap restriction.
• Bad advice in docs for octal mode #60773 — corrected the misleading "must quote octal modes" warning.
• service.running doesn't handle special characters in sig #34929 salt.states.service.running — documented sig matching
  against shell special characters.
• [BUG] service.running doesn't trigger systemctl daemon-reload as expected #57606 / [DOCS] salt.states.service.running confusing discrepencies #60784 / [DOCS] states.service.running example does not match the documentation #63852 salt.states.service.running — documented
  the reload and full_restart mod-watch options, and added a
  note about systemd daemon-reload ordering.
• cmd.run's runas does not drop privileges #57505 / Update Documentation For cmd.run regarding Windows runas argument #57949 salt.modules.cmdmod.run — documented runas
  group preservation on Linux and the Windows password requirement.
• [DOCS] user.present features not supported on Windows #61689 salt.states.user.present — flagged Windows-unsupported
  uid, gid, allow_uid_change, allow_gid_change.
• [DOCS] salt.states.win_pki available stores #64021 salt.states.win_pki — listed the available certificate
  stores and how to enumerate them with PowerShell.
• clarification for netmiko options: keepalive & always_alive #56182 salt.proxy.netmiko_px — distinguished keepalive (in-band
  KEEPALIVE packets) from always_alive (connection lifecycle).
• Copy+paste errors in postgres docs #51213 salt.states.postgres_privileges — fixed copy-pasted
  maintenance_db description.
• salt.pillar.file_tree documentation is wrong #57405 salt.pillar.file_tree — corrected the example so the
  rendered pillar tree matches the on-disk layout.
• [DOCS] salt.modules.saltcheck has duplicate examples and a confusing definition #63364 salt.modules.saltcheck — removed a duplicated jinja/setup
  example block; clarified the assertion definition.
• [TECH DEBT] file.chown changes the ownership of broken symbolic links #61405 salt.modules.file.chown — documented the broken-symlink
  lchown fallback.
• [DOCS] jobs.last_run has the wrong description and no paramter documentation #60406 salt.runners.jobs.last_run — rewrote the description and
  added parameter documentation.
• docker_container.running state - command attribute does not support lists #55881 salt.states.docker_container.running — documented that
  command accepts either a string or a YAML list.
• [BUG] docker_image.present interface does not match docs #56956 salt.states.docker_image.present — clarified that sls
  accepts a comma-separated string only (not a YAML list).
• [DOCS] docker_container.running hostname is not set from name #66409 salt.states.docker_container.running — corrected the
  hostname fallback wording.

Branches

Per-branch PR — each release branch is being shipped separately because
the affected docstrings have drifted independently across the lines.

issue group	3006.x	3007.x	3008.x	master
modules / states / runners / wheel / pillar (above)	yes	yes	yes	yes
docker_container / docker_image / aptpkg / netmiko_px / ddns	yes	yes	n/a (extracted)	n/a (extracted)

Out of scope (issues from the original list that are no longer

applicable on this branch and need no doc change)

• [DOCS] linux_acl.list_absent does not support "perms" #66891 linux_acl.list_absent perms — already removed from the
  docstring on every active branch.
• [DOCS] still many references to "SaltStack, Inc" in source #61374 "SaltStack, Inc" references — git grep finds zero hits
  inside salt/ and tests/; the remaining hits are in
  AUTHORS, the Windows installer and a tutorial RST file that PR9
  is not allowed to touch.
• [DOCS] References to keystore module, keystore does not exist #67145 keystore references — the module itself still exists on
  3006.x / 3007.x and has been removed entirely on 3008.x / master,
  so the docs are correct on each branch.
• [BUG] Default values in docstring parameters have no space after parameter #57053 — the missing-space param:default formatting reported
  6 years ago is no longer present in salt/modules/state.py.
• [DOCS] virtualname should be listed in all modules. without having to guess.  #61366 (universal __virtualname__ listing) — handled for the
  modules that are also flagged by other linked issues
  (groupadd → group, aptpkg → apt, virtualenv_mod → virtualenv). A blanket sweep across the remaining ~50 modules
  is deferred to a follow-up PR to keep this one reviewable.
• [DOCS] Witing grains guide does not mention the grains argument #66628 (custom-grains topic guide), [DOCS] state tutorial pt3 -> use of deprecated argument gid_from_name #65939 (states_pt3 tutorial),
  DDNS Module configuration structure. #52650 (DDNS minion-config layout) and [DOCS] Confusion with "kwarg" vs. "kwargs" #64283 (Salt-API kwarg
  schema) all want changes inside doc/**.rst files; PR9 is
  docstring-only by charter — those are deferred to PR8 (states) /
  PR5 (pillar) RST passes.

Test plan

• CI green on the per-branch base.
• pre-commit run --all-files clean on the touched files
  (verified locally with isolated PRE_COMMIT_HOME).
• make -C doc html produces no Sphinx warnings on the rendered
  pages for each touched module.
• python -c "help(salt.modules.<m>.<f>)" round-trips for each
  edited function.

🤖 Generated with Claude Code (Claude Opus 4.7).