Skip to content

Conversation

@haug1
Copy link
Contributor

@haug1 haug1 commented Oct 17, 2025

1366 lines of code, zero coffee breaks. Thanks Claude.

Only completely human interaction:
This was almost completely AI-driven and AI-generated PR. I've NOT done my due diligence testing and reviewing this code properly. This is an experiment I've done after getting my hands on Claude Code driven by the 4.5 sonnet model (I think?). I hope it can give real-life value, otherwise I bet it will still be interesting to some people

1366 lines of code, zero coffee breaks. Thanks Claude.
@haug1 haug1 changed the title feat(custom): add shared exec worker backend for modules (AI) feat(custom): add shared exec worker backend for modules Oct 17, 2025
@haug1 haug1 closed this Oct 17, 2025
@haug1 haug1 reopened this Oct 17, 2025
@haug1
Copy link
Contributor Author

haug1 commented Oct 22, 2025

I've looked more at the code and I think it looks pretty good. It's a requested feature implemented as a new opt-in configuration flag under the custom module. To me the changes seem fully backward-compatible. Would you care to have a quick look at this, @Alexays?

I'm willing to spend the time finishing this if I get some help, some quick description what needs to change in order to get this merged!
Here's changes I already think is needed:

  • remove lots of the documentation markdown files. (They were primarily meant to document the workflow of this PR and working with AI/Claude Code)
  • add actual documentation of the new feature in the man files

## Time Investment

- **Human active time:** ~15 minutes (task formulation with ChatGPT, approvals, testing, PR prep)
- **AI processing time:** ~40 minutes (analysis, design, implementation, documentation)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uhh, this number seems exaggerated and might include idling time of just having the client open and not actually working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant