Skip to content

Commit

Permalink
Skip files that include Runme session IDs (those are output files) (#459
Browse files Browse the repository at this point in the history
)

* Skip files that include Runme session IDs (those are output files)

* Add test coverage

* Bring back attaching to dlv

* Address review suggestions
  • Loading branch information
sourishkrout authored Dec 29, 2023
1 parent 68a676d commit 9fe054f
Show file tree
Hide file tree
Showing 4 changed files with 36 additions and 1 deletion.
11 changes: 10 additions & 1 deletion .vscode/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,15 @@
"--tls",
"/tmp/runme/tls",
]
}
},
{
"name": "Connect to dlv",
"type": "go",
"request": "attach",
"mode": "remote",
"remotePath": "${workspaceFolder}",
"port": 56379,
"host": "127.0.0.1"
},
]
}
5 changes: 5 additions & 0 deletions internal/project/project.go
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,11 @@ func getCodeBlocksFromFile(path string) (document.CodeBlocks, error) {
func getCodeBlocks(data []byte) (document.CodeBlocks, error) {
identityResolver := identity.NewResolver(identity.DefaultLifecycleIdentity)
d := document.New(data, identityResolver)

if f, err := d.Frontmatter(); err == nil && f != nil && f.Runme.Session.ID != "" {
return nil, nil
}

node, err := d.Root()
if err != nil {
return nil, err
Expand Down
6 changes: 6 additions & 0 deletions internal/project/project_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,7 @@ func TestProjectLoad(t *testing.T) {
LoadEventFoundDir, // "."
LoadEventFoundFile, // "ignored.md"
LoadEventFoundFile, // "readme.md"
LoadEventFoundFile, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
LoadEventFinishedWalk,
LoadEventStartedParsingDocument, // "ignored.md"
LoadEventFinishedParsingDocument, // "ignored.md"
Expand All @@ -317,6 +318,8 @@ func TestProjectLoad(t *testing.T) {
LoadEventFinishedParsingDocument, // "readme.md"
LoadEventFoundTask, // unnamed; echo-hello
LoadEventFoundTask, // named; my-task
LoadEventStartedParsingDocument, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
LoadEventFinishedParsingDocument, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
}
require.EqualValues(
t,
Expand Down Expand Up @@ -350,11 +353,14 @@ func TestProjectLoad(t *testing.T) {
LoadEventStartedWalk,
LoadEventFoundDir, // "."
LoadEventFoundFile, // "readme.md"
LoadEventFoundFile, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
LoadEventFinishedWalk,
LoadEventStartedParsingDocument, // "readme.md"
LoadEventFinishedParsingDocument, // "readme.md"
LoadEventFoundTask, // unnamed; echo-hello
LoadEventFoundTask, // named; my-task
LoadEventStartedParsingDocument, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
LoadEventFinishedParsingDocument, // "session-01HJS35FZ2K0JBWPVAXPMMVTGN.md"
}
require.EqualValues(
t,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
---
runme:
id: 01HJS33TJYXZ6KJPG2SZZ6D1H5
version: v2.0
session:
id: 01HJS35FZ2K0JBWPVAXPMMVTGN
updated: 2023-12-28 15:16:06-05:00
---

```sh {"name":"iam-session","id":"01HJS341EYAN1QPKK7D835C6FD"}
echo "i am a session outputs file"

# Ran on 2023-12-28 15:52:03-05:00 for 719ms exited with 0
i am a session outputs file
```

0 comments on commit 9fe054f

Please sign in to comment.