max-mapper / max-mapper/monu

Extend process execution definition

Open
#20 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
CSS
Stars
1.1k
Forks
69
PR merge metrics
No merged PRs in 30d

Description

Currently there is simple format to add new process

  "processes": {
    "web-1": "cd /some/dir;sudo http-server . -p 8081",
  }

This is fine and it's good idea to keep definition simple, but to have config future proof, it would be great if monu can support also object as process definition.

  "processes": {
    "web-1": {
      "command": "http-server . -p 8081",
      "cwd": "/some/dir",
      "sudo": true
    }
  }

I took exec options as inspiration.

I think this will open more possibilities for adding new processes using interactive form rather than editing config file, also it will make config file much more readable.

What do you think?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the code that parses the processes configuration and launches commands, then compare its current behavior with Node.js child_process exec options. Done means both the existing string form and the proposed object form are accepted, including command, cwd, and sudo.

Written by the indexing model from the issue text.

Assessment

Tech stack
node.js
Domain
desktop
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.