pam/adapter: Avoid using pointer receivers for Init functions

And other functions where we don't write to the models

Author: 3v1n0

pam/internal/adapter/authentication.go

@@ -174,7 +174,7 @@ func newAuthenticationModel(client authd.PAMClient, clientType PamClientType) au
 }
 
 // Init initializes authenticationModel.
-func (m *authenticationModel) Init() tea.Cmd {
+func (m authenticationModel) Init() tea.Cmd {
 	return nil
 }
 

pam/internal/adapter/authmodeselection.go

@@ -57,7 +57,7 @@ func newAuthModeSelectionModel(clientType PamClientType) authModeSelectionModel
 }
 
 // Init initializes authModeSelectionModel.
-func (m *authModeSelectionModel) Init() tea.Cmd {
+func (m authModeSelectionModel) Init() tea.Cmd {
 	if m.clientType != InteractiveTerminal {
 		// This is handled by the GDM or Native model!
 		return nil

pam/internal/adapter/userselection.go

@@ -58,7 +58,7 @@ func newUserSelectionModel(pamMTx pam.ModuleTransaction, clientType PamClientTyp
 }
 
 // Init initializes userSelectionModel.
-func (m *userSelectionModel) Init() tea.Cmd {
+func (m userSelectionModel) Init() tea.Cmd {
 	return nil
 }