Skip to content

Commit

Permalink
remote: neaten MonadStore
Browse files Browse the repository at this point in the history
  • Loading branch information
sorki committed Nov 30, 2023
1 parent ffb2905 commit c578cf6
Showing 1 changed file with 7 additions and 9 deletions.
16 changes: 7 additions & 9 deletions hnix-store-remote/src/System/Nix/Store/Remote/MonadStore.hs
Original file line number Diff line number Diff line change
Expand Up @@ -10,23 +10,22 @@ module System.Nix.Store.Remote.MonadStore
, MonadRemoteStore0
, MonadRemoteStore
, MonadRemoteStoreHandshake
-- *
-- * Reader helpers
, getStoreDir
, getStoreSocket
, getProtoVersion
-- *
-- * Logs
, appendLogs
, getLogs
, flushLogs
, gotError
, getErrors
-- *
-- * Data required from client
, getData
, setData
, clearData
) where


import Control.Monad.Except (MonadError)
import Control.Monad.IO.Class (MonadIO)
import Control.Monad.Reader (MonadReader, ask)
Expand Down Expand Up @@ -81,8 +80,7 @@ newtype RemoteStoreT r m a = RemoteStoreT
, Applicative
, Monad
, MonadReader r
--, MonadState StoreState -- Avoid making the internal state explicit
--, MonadFail
--, MonadState StoreState -- Avoid making the internal state explicit
, MonadError RemoteStoreError
, MonadIO
)
Expand Down Expand Up @@ -140,14 +138,14 @@ getStoreSocket = hasStoreSocket <$> RemoteStoreT ask
getProtoVersion :: HasProtoVersion r => MonadRemoteStore0 r ProtoVersion
getProtoVersion = hasProtoVersion <$> RemoteStoreT ask

-- * Logs

gotError :: MonadRemoteStore0 r Bool
gotError = any isError <$> getLogs

getErrors :: MonadRemoteStore0 r [Logger]
getErrors = filter isError <$> getLogs

-- *

appendLogs :: [Logger] -> MonadRemoteStore0 r ()
appendLogs x = RemoteStoreT
$ modify
Expand All @@ -159,7 +157,7 @@ getLogs = remoteStoreState_logs <$> RemoteStoreT get
flushLogs :: MonadRemoteStore0 r ()
flushLogs = RemoteStoreT $ modify $ \s -> s { remoteStoreState_logs = mempty }

-- *
-- * Data required from client

getData :: MonadRemoteStore0 r (Maybe ByteString)
getData = remoteStoreState_mData <$> RemoteStoreT get
Expand Down

0 comments on commit c578cf6

Please sign in to comment.