Skip to content

Commit 2255d14

Browse files
sourishkroutclaude
andauthored
fix: add getPty method to inlineCommand (#1080)
## Summary - Adds the missing `getPty()` method to `inlineCommand`, delegating to the underlying `internalCommand` via the `commandWithPty` interface - Follows the same pattern already used by `inlineShellCommand` and `terminalCommand` - Enables PTY access for inline commands that need it ## Test plan - [ ] Verify `runme run lint` passes - [ ] Verify `runme run test` passes - [ ] Confirm inline commands can access PTY when backed by a PTY-capable internal command 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Signed-off-by: Sebastian (Tiedtke) Huckleberry <sebastiantiedtke@gmail.com> Co-authored-by: Claude Opus 4 <noreply@anthropic.com>
1 parent 8470e7b commit 2255d14

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

.release-notes-v3.16.10.md

Lines changed: 0 additions & 5 deletions
This file was deleted.

command/command_inline.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package command
33
import (
44
"bytes"
55
"context"
6+
"os"
67

78
"go.uber.org/zap"
89
)
@@ -13,6 +14,14 @@ type inlineCommand struct {
1314
logger *zap.Logger
1415
}
1516

17+
func (c *inlineCommand) getPty() *os.File {
18+
cmd, ok := c.internalCommand.(commandWithPty)
19+
if !ok {
20+
return nil
21+
}
22+
return cmd.getPty()
23+
}
24+
1625
func (c *inlineCommand) Start(ctx context.Context) error {
1726
buf := new(bytes.Buffer)
1827
bw := bulkWriter{Writer: buf}

0 commit comments

Comments
 (0)