@DevelopmentCats can you please review this PR

I am also wondering if it would be worth having the module output something that could be used by a coder_app resource to provide a dynamic live preview based on the detected project type.

I am also wondering if it would be worth having the module output something that could be used by a coder_app resource to provide a dynamic live preview based on the detected project type.

@DevelopmentCats so whats on your mind

I am also wondering if it would be worth having the module output something that could be used by a coder_app resource to provide a dynamic live preview based on the detected project type.

@DevelopmentCats so whats on your mind

You already have a check for verifying that there is a running app as a result of the module running at the end of run.sh

I think what you should do is take this and set the project port to a local which you could then use in a coder app resource to set a preview app for this module.

You could also set a variable to disable this feature in the module if you would like, but I think this would fit perfectly in this module with this logic.

IE..

resource "coder_app" "preview" {
  count        = var.enable_preview_app ? 1 : 0
  agent_id     = coder_agent.dev.id
  display_name = "Live Preview"
  url          = "http://localhost:${local.detected_port}"
  # other configuration...
}

@DevelopmentCats

Screencast.from.2025-08-13.22-41-52.mp4
@DevelopmentCats

This looks great! the only thing you might want to point the coder app icon to an actual existing icon or emoji in coder.
@MAVRICK-1

@DevelopmentCats thankyou very much for your help

@DevelopmentCats thankyou very much for your help

No Problemo.

Sorry but can you run bun run fmt one last time? @MAVRICK-1

Okay 👍

@DevelopmentCats done the changes

Once we have these last comments addressed we can go ahead and get this merged @MAVRICK-1

@DevelopmentCats done

@DevelopmentCats just a headsup

@matifali test are failing in other module , can we get this merge ?

@matifali test are failing in other module , can we get this merge ?

re-running the ci now

Ci passed

Ci passed

@MAVRICK-1 I see but can you do that one last thing I mentioned above, and then Ill merge

@DevelopmentCats what thing? I have addressed all the reviews

@DevelopmentCats what thing?

My comment above about adding the version field to the rest of the examples in the README

@DevelopmentCats done ✅✅👍