Skip to content

Update module reconfiguration lifecycle docs#5042

Open
shannonbradshaw wants to merge 1 commit into
mainfrom
claude/update-module-reconfigure-lifecycle
Open

Update module reconfiguration lifecycle docs#5042
shannonbradshaw wants to merge 1 commit into
mainfrom
claude/update-module-reconfigure-lifecycle

Conversation

@shannonbradshaw
Copy link
Copy Markdown
Collaborator

Source changes

Docs changes

  • docs/build-modules/module-reference.md: Removed the "In-place reconfigure" row from the Python/Go defaults table (Python's reconfigure() via the Reconfigurable protocol is no longer called by viam-server). Added a paragraph clarifying that viam-server always rebuilds modular resources. Updated the ReconfigureResource RPC description to reflect it now rebuilds rather than updates in place.
  • static/include/components/apis/overrides/protos/sensor.Reconfigure.md: Updated the override description to state that modular resources are rebuilt rather than reconfigured atomically in place.

How I found these

  • Xref lookup: search-patterns.yaml entry for Resource.Reconfigure (discovered 2026-04-15)
  • Grep matches: grep -rl "Reconfigure" docs/build-modules/ found 4 pages; the module-reference.md table had the incorrect Python claim

Generated by daily docs change agent


Generated by Claude Code

viam-server now always rebuilds modular resources (close + re-create)
when their configuration changes. In-place reconfiguration via
Reconfigure() is no longer supported for modular resources in any
language. This removes the incorrect claim that Python modules can
implement reconfigure() to handle config changes in place.

Source: viamrobotics/rdk#5944

https://claude.ai/code/session_01Xd7GAtzjRzypiJK1amk4Uu
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit efe2b4f
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/69fb06e98dd93c0008248d87
😎 Deploy Preview https://deploy-preview-5042--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 37 (🔴 down 12 from production)
Accessibility: 99 (no change from production)
Best Practices: 83 (no change from production)
SEO: 92 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label May 6, 2026
Copy link
Copy Markdown
Collaborator Author

Hold: source change reverted. The RDK change this PR documents (viamrobotics/rdk#5944, Remove-Reconfigure) was reverted by viamrobotics/rdk#5997 on 2026-05-07. Reconfigure is back on the Resource interface temporarily while the cascade-fix bakes on main.

The #5997 PR description says: "To reland Remove-Reconfigure: revert this PR." So Remove-Reconfigure is expected to re-land in 1-2 weeks.

Recommendation: Do not merge this PR until Remove-Reconfigure re-lands. Once it does, this PR's changes will be correct again and can be merged (possibly with minor updates if the re-land differs from #5944).


Generated by daily docs change agent


Generated by Claude Code

@viamrobotics-overwatch
Copy link
Copy Markdown

Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

Auto-comment from overwatch. Will not re-nudge for 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants