Summary

During the migration we reduced revive to only indent-error-flow, so exported identifiers are no longer checked for missing documentation or unused parameters/receivers. Previously rules like exported, var-naming, and unused-parameter kept APIs tidy.

Desired outcome

• Revisit the revive configuration and re-enable the useful rules we had before (e.g., exported, var-naming, unused-parameter, unused-receiver, doc comments).
• Fix or suppress any violations appropriately.
• Ensure revive passes cleanly afterwards.

Notes

• Rules can be scoped to specific packages if necessary.
• Avoid blanket disables; the goal is to keep exported APIs well documented and consistent.