@@ -45,12 +45,12 @@ type API struct {
4545
4646// lockCh protects the below fields. We use a channel instead of a
4747// mutex so we can handle cancellation properly.
48- lockCh chan struct {}
49- containers codersdk.WorkspaceAgentListContainersResponse
50- mtime time.Time
51- devcontainerNames map [string ]struct {}// Track devcontainer names to avoid duplicates.
52- knownDevcontainers []codersdk.WorkspaceAgentDevcontainer // Track predefined and runtime-detected devcontainers.
53- configModifiedTimes map [string ]time.Time // Track when config files were last modified.
48+ lockCh chan struct {}
49+ containers WorkspaceAgentListContainersResponse
50+ mtime Time
51+ devcontainerNames map [string ]struct {}// Track devcontainer names to avoid duplicates.
52+ knownDevcontainers WorkspaceAgentDevcontainer // Track predefined and runtime-detected devcontainers.
53+ configFileModifiedTimes map [string ]time.Time // Track when config files were last modified.
5454}
5555
5656// Option is a functional option for API.
@@ -108,16 +108,16 @@ func WithWatcher(w watcher.Watcher) Option {
108108func NewAPI (logger slog.Logger ,options ... Option )* API {
109109ctx ,cancel := context .WithCancel (context .Background ())
110110api := & API {
111- ctx :ctx ,
112- cancel :cancel ,
113- done :make (chan struct {}),
114- logger :logger ,
115- clock :quartz .NewReal (),
116- cacheDuration :defaultGetContainersCacheDuration ,
117- lockCh :make (chan struct {},1 ),
118- devcontainerNames :make (map [string ]struct {}),
119- knownDevcontainers : []codersdk.WorkspaceAgentDevcontainer {},
120- configModifiedTimes :make (map [string ]time.Time ),
111+ ctx :ctx ,
112+ cancel :cancel ,
113+ done :make (chan struct {}),
114+ logger :logger ,
115+ clock :quartz .NewReal (),
116+ cacheDuration :defaultGetContainersCacheDuration ,
117+ lockCh :make (chan struct {},1 ),
118+ devcontainerNames :make (map [string ]struct {}),
119+ knownDevcontainers :WorkspaceAgentDevcontainer {},
120+ configFileModifiedTimes :make (map [string ]time.Time ),
121121}
122122for _ ,opt := range options {
123123opt (api )
@@ -281,7 +281,7 @@ func (api *API) getContainers(ctx context.Context) (codersdk.WorkspaceAgentListC
281281// Check if this container was created after the config
282282// file was modified.
283283if configFile != "" && api .knownDevcontainers [knownIndex ].Dirty {
284- lastModified ,hasModTime := api .configModifiedTimes [configFile ]
284+ lastModified ,hasModTime := api .configFileModifiedTimes [configFile ]
285285if hasModTime && container .CreatedAt .After (lastModified ) {
286286api .logger .Info (ctx ,"clearing dirty flag for container created after config modification" ,
287287slog .F ("container" ,container .ID ),
@@ -316,7 +316,7 @@ func (api *API) getContainers(ctx context.Context) (codersdk.WorkspaceAgentListC
316316
317317dirty := dirtyStates [workspaceFolder ]
318318if dirty {
319- lastModified ,hasModTime := api .configModifiedTimes [configFile ]
319+ lastModified ,hasModTime := api .configFileModifiedTimes [configFile ]
320320if hasModTime && container .CreatedAt .After (lastModified ) {
321321api .logger .Info (ctx ,"new container created after config modification, not marking as dirty" ,
322322slog .F ("container" ,container .ID ),
@@ -473,7 +473,7 @@ func (api *API) markDevcontainerDirty(configPath string, modifiedAt time.Time) {
473473}
474474
475475// Record the timestamp of when this configuration file was modified.
476- api .configModifiedTimes [configPath ]= modifiedAt
476+ api .configFileModifiedTimes [configPath ]= modifiedAt
477477
478478for i := range api .knownDevcontainers {
479479if api .knownDevcontainers [i ].ConfigPath == configPath {