feat: Added stdin option

Author: kevinwang5658

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "codify-plugin-lib",
-  "version": "1.0.182-beta9",
+  "version": "1.0.182-beta10",
   "description": "Library plugin library",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
@@ -22,7 +22,7 @@
     "ajv": "^8.12.0",
     "ajv-formats": "^2.1.1",
     "clean-deep": "^3.4.0",
-    "codify-schemas": "1.0.86-beta4",
+    "codify-schemas": "1.0.86-beta5",
     "lodash.isequal": "^4.5.0",
     "nanoid": "^5.0.9",
     "strip-ansi": "^7.1.0",

package.json

@@ -28,9 +28,10 @@ export enum SpawnStatus {
  */
 export interface SpawnOptions {
   cwd?: string;
-  env?: Record<string, unknown>,
-  interactive?: boolean,
-  requiresRoot?: boolean,
+  env?: Record<string, unknown>;
+  interactive?: boolean;
+  requiresRoot?: boolean;
+  stdin?: boolean;
 }
 
 export class SpawnError extends Error {

src/pty/index.ts

@@ -34,8 +34,12 @@ export class SequentialPty implements IPty {
   }
 
   async spawnSafe(cmd: string, options?: SpawnOptions): Promise<SpawnResult> {
+    if (cmd.includes('sudo')) {
+      throw new Error('Do not directly use sudo. Use the option { requiresRoot: true } instead')
+    }
+
     // If sudo is required, we must delegate to the main codify process.
-    if (options?.interactive || options?.requiresRoot) {
+    if (options?.stdin || options?.requiresRoot) {
       return this.externalSpawn(cmd, options);
     }
 
@@ -59,8 +63,10 @@ export class SequentialPty implements IPty {
       const initialCols = process.stdout.columns ?? 80;
       const initialRows = process.stdout.rows ?? 24;
 
+      const args = options?.interactive ? ['-i', '-c', cmd] : ['-c', cmd]
+
       // Run the command in a pty for interactivity
-      const mPty = pty.spawn(this.getDefaultShell(), ['-c', cmd], {
+      const mPty = pty.spawn(this.getDefaultShell(), args, {
         ...options,
         cols: initialCols,
         rows: initialRows,