bash-dap 0.2__tar.gz → 0.3__tar.gz

{bash_dap-0.2 → bash_dap-0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bash-dap
-Version: 0.2
+Version: 0.3
 Summary: A debug adapter (Debug Adapter Protocol) for bash scripts, written in bash and Python
 Author-email: Ilia Maslakov <il.smind@gmail.com>
 License: GPL-3.0-or-later
@@ -18,9 +18,9 @@ Dynamic: license-file
 # bash-dap
 
 A debug adapter of the [Debug Adapter Protocol](https://microsoft.github.io/debug-adapter-protocol/)
-for bash scripts: breakpoints on lines and on functions, steps into, over and out of functions,
-the call stack, the
-variables (arrays and associative arrays as trees), expressions, pause.
+for bash scripts: breakpoints on lines, with conditions, and on functions, steps into, over and
+out of functions, the call stack, the variables (arrays and associative arrays as trees),
+expressions, pause.
 
 It needs nothing but bash 4.1 or newer and Python 3.8 or newer: no bashdb, no Node.js.  (The bash
 of macOS is 3.2: a newer one, from Homebrew say, is given to the launch as *bash*.)  The debugger itself is a script
@@ -80,12 +80,16 @@ A breakpoint on a function stops at its first command, at each call, in a subshe
 what the script defines later, or a file it sources, is found as well, as the name is all
 the debugger looks for.
 
+The condition of a breakpoint is a command of bash, `[[ $f == report.pdf ]]` or
+`(( count > 2 ))`: the script stops there when its status is 0.  It runs in a subshell, so it
+sees the variables of the script and changes none of them.
+
 When the session ends the adapter ends the script by its process group, the session it has to
 itself; never by a parent, never anything else.
 
 ## Not yet
 
-Conditional breakpoints, a script that another script runs with a new bash.
+A script that another script runs with a new bash.
 
 ## License
 

{bash_dap-0.2 → bash_dap-0.3}/README.md

@@ -1,9 +1,9 @@
 # bash-dap
 
 A debug adapter of the [Debug Adapter Protocol](https://microsoft.github.io/debug-adapter-protocol/)
-for bash scripts: breakpoints on lines and on functions, steps into, over and out of functions,
-the call stack, the
-variables (arrays and associative arrays as trees), expressions, pause.
+for bash scripts: breakpoints on lines, with conditions, and on functions, steps into, over and
+out of functions, the call stack, the variables (arrays and associative arrays as trees),
+expressions, pause.
 
 It needs nothing but bash 4.1 or newer and Python 3.8 or newer: no bashdb, no Node.js.  (The bash
 of macOS is 3.2: a newer one, from Homebrew say, is given to the launch as *bash*.)  The debugger itself is a script
@@ -63,12 +63,16 @@ A breakpoint on a function stops at its first command, at each call, in a subshe
 what the script defines later, or a file it sources, is found as well, as the name is all
 the debugger looks for.
 
+The condition of a breakpoint is a command of bash, `[[ $f == report.pdf ]]` or
+`(( count > 2 ))`: the script stops there when its status is 0.  It runs in a subshell, so it
+sees the variables of the script and changes none of them.
+
 When the session ends the adapter ends the script by its process group, the session it has to
 itself; never by a parent, never anything else.
 
 ## Not yet
 
-Conditional breakpoints, a script that another script runs with a new bash.
+A script that another script runs with a new bash.
 
 ## License
 

{bash_dap-0.2 → bash_dap-0.3}/bash_dap/__init__.py

@@ -1,3 +1,3 @@
 """bash-dap: a debug adapter of the Debug Adapter Protocol for bash scripts."""
 
-__version__ = "0.2"
+__version__ = "0.3"

{bash_dap-0.2 → bash_dap-0.3}/bash_dap/adapter.py

@@ -199,8 +199,11 @@ class Adapter:
             return
         with open(os.path.join(self.dir, "breaks"), "w") as f:
             for path, lines in self.breaks.items():
-                for line in lines:
-                    f.write("%s:%d\n" % (path, line))
+                for line, condition in lines:
+                    if condition:
+                        f.write("%s:%d\t%s\n" % (path, line, condition))
+                    else:
+                        f.write("%s:%d\n" % (path, line))
             for name in self.functions:
                 f.write("%s()\n" % name)
         self.gen += 1
@@ -297,6 +300,7 @@ class Adapter:
         self.respond(request, {
             "supportsConfigurationDoneRequest": True,
             "supportsFunctionBreakpoints": True,
+            "supportsConditionalBreakpoints": True,
             "supportsEvaluateForHovers": True,
             "supportsTerminateRequest": True,
         })
@@ -355,12 +359,14 @@ class Adapter:
     def on_setBreakpoints(self, request):
         args = request.get("arguments", {})
         path = os.path.realpath(args.get("source", {}).get("path", ""))
-        lines = [b["line"] for b in args.get("breakpoints", []) if "line" in b]
+        # a line and its condition, a command of bash whose status says whether to stop
+        lines = [(b["line"], " ".join(b.get("condition", "").split()))
+                 for b in args.get("breakpoints", []) if "line" in b]
         self.breaks[path] = lines
         self.save_breaks()
         self.respond(request, {"breakpoints": [
             {"id": i + 1, "verified": True, "line": line, "source": {"path": path}}
-            for i, line in enumerate(lines)]})
+            for i, (line, _) in enumerate(lines)]})
 
     def on_setFunctionBreakpoints(self, request):
         """The functions to stop in at their first command; a name that is no name of a

{bash_dap-0.2 → bash_dap-0.3}/bash_dap/harness.sh

@@ -17,7 +17,9 @@
 # the script is stopped.  At any time the adapter may write the breakpoints to the file "breaks",
 # FILE:LINE a line, then make the empty file "gen.N" of the next generation N; and stop the
 # script at its next command by writing "step 0" to the file "state".  A line NAME() of
-# "breaks" is a breakpoint on the function NAME: the script stops at its first command.
+# "breaks" is a breakpoint on the function NAME: the script stops at its first command.  After
+# FILE:LINE and a tab, a command of bash is the condition of the breakpoint: the script stops
+# there only when its status is 0.
 #
 # A subshell is a process of its own: what it is told, a step, a breakpoint, goes to the others
 # through these files.  The names of the debugger begin with __bdap_.
@@ -35,8 +37,8 @@ __bdap_mode=run
 __bdap_depth=0
 # the breakpoints by FILE:LINE, and the generation of the file they came from; __bdap_names has
 # NAME:LINE, the name of the file only, that the trap tests at each command; __bdap_funcs has
-# NAME() of the functions to stop in
-declare -A __bdap_breaks=() __bdap_names=() __bdap_funcs=()
+# NAME() of the functions to stop in; __bdap_conds the conditions of FILE:LINE
+declare -A __bdap_breaks=() __bdap_names=() __bdap_funcs=() __bdap_conds=()
 __bdap_gen=0
 __bdap_inside=
 __bdap_prev=  # the place of the command before: a line stops when it is come to, not again
@@ -98,8 +100,13 @@ __bdap_breaks_load () {
     __bdap_breaks=()
     __bdap_names=()
     __bdap_funcs=()
-    while read -r __bdap_place; do
+    __bdap_conds=()
+    while IFS= read -r __bdap_place; do
         [[ -z $__bdap_place ]] && continue
+        if [[ $__bdap_place == *$'\t'* ]]; then
+            __bdap_conds["${__bdap_place%%$'\t'*}"]=${__bdap_place#*$'\t'}
+            __bdap_place=${__bdap_place%%$'\t'*}
+        fi
         if [[ $__bdap_place == *"()" ]]; then
             __bdap_funcs["$__bdap_place"]=1
             continue
@@ -148,6 +155,13 @@ __bdap_on_line () {
     [[ -z $__bdap_word || ${__bdap_text[0]-} == *"$__bdap_word"* ]]
 }
 
+# Whether the condition of a breakpoint holds, none always: run in a subshell, it sees the
+# variables of the script and changes none of them
+__bdap_condition () {
+    [[ -z $1 ]] && return 0
+    ( eval "$1" ) </dev/null >/dev/null 2>&1
+}
+
 # Before a command of the script that may stop: what the script has set, -e, -u, -x, is not
 # the debugger's, which runs without
 __bdap_hook () {
@@ -179,7 +193,8 @@ __bdap_check () {
     next) ((__bdap_d <= __bdap_depth)) && __bdap_reason=step ;;
     finish) ((__bdap_d < __bdap_depth)) && __bdap_reason=step ;;
     esac
-    if [[ -z $__bdap_reason && -n ${__bdap_breaks["$__bdap_file:$__bdap_line"]-} ]] && __bdap_on_line; then
+    if [[ -z $__bdap_reason && -n ${__bdap_breaks["$__bdap_file:$__bdap_line"]-} ]] && __bdap_on_line \
+        && __bdap_condition "${__bdap_conds["$__bdap_file:$__bdap_line"]-}"; then
         __bdap_reason=breakpoint
     fi
     # the first command of a function to stop in: the calls are deeper than at the command

{bash_dap-0.2 → bash_dap-0.3}/bash_dap.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bash-dap
-Version: 0.2
+Version: 0.3
 Summary: A debug adapter (Debug Adapter Protocol) for bash scripts, written in bash and Python
 Author-email: Ilia Maslakov <il.smind@gmail.com>
 License: GPL-3.0-or-later
@@ -18,9 +18,9 @@ Dynamic: license-file
 # bash-dap
 
 A debug adapter of the [Debug Adapter Protocol](https://microsoft.github.io/debug-adapter-protocol/)
-for bash scripts: breakpoints on lines and on functions, steps into, over and out of functions,
-the call stack, the
-variables (arrays and associative arrays as trees), expressions, pause.
+for bash scripts: breakpoints on lines, with conditions, and on functions, steps into, over and
+out of functions, the call stack, the variables (arrays and associative arrays as trees),
+expressions, pause.
 
 It needs nothing but bash 4.1 or newer and Python 3.8 or newer: no bashdb, no Node.js.  (The bash
 of macOS is 3.2: a newer one, from Homebrew say, is given to the launch as *bash*.)  The debugger itself is a script
@@ -80,12 +80,16 @@ A breakpoint on a function stops at its first command, at each call, in a subshe
 what the script defines later, or a file it sources, is found as well, as the name is all
 the debugger looks for.
 
+The condition of a breakpoint is a command of bash, `[[ $f == report.pdf ]]` or
+`(( count > 2 ))`: the script stops there when its status is 0.  It runs in a subshell, so it
+sees the variables of the script and changes none of them.
+
 When the session ends the adapter ends the script by its process group, the session it has to
 itself; never by a parent, never anything else.
 
 ## Not yet
 
-Conditional breakpoints, a script that another script runs with a new bash.
+A script that another script runs with a new bash.
 
 ## License
 

{bash_dap-0.2 → bash_dap-0.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "bash-dap"
-version = "0.2"
+version = "0.3"
 description = "A debug adapter (Debug Adapter Protocol) for bash scripts, written in bash and Python"
 readme = "README.md"
 license = { text = "GPL-3.0-or-later" }

{bash_dap-0.2 → bash_dap-0.3}/tests/test_adapter.py

@@ -223,6 +223,27 @@ class SessionTest(unittest.TestCase):
         c.request("continue", {"threadId": 1})
         self.assertEqual(c.wait_event("exited")["body"]["exitCode"], 3)
 
+    def test_conditional_breakpoint(self):
+        c = self.client = Client()
+        c.request("initialize", {"adapterID": "bash-dap", "linesStartAt1": True})
+        c.request("launch", {"program": script("backup.sh"), "cwd": SCRIPTS})
+        c.wait_event("initialized")
+        response = c.request("setBreakpoints", {"source": {"path": script("backup.sh")},
+                                                "breakpoints": [
+            {"line": 15, "condition": "[[ $f == photo.jpg ]]"},
+            # an assignment in a condition changes nothing of the script
+            {"line": 16, "condition": "total=1000; false"}]})
+        self.assertTrue(all(b["verified"] for b in response["body"]["breakpoints"]))
+        c.request("configurationDone")
+        self.assertEqual(c.wait_event("stopped")["body"]["reason"], "breakpoint")
+        self.assertEqual(c.stop_place(), [("main", "backup.sh", 15)])
+        self.assertEqual(c.evaluate("$f")["body"]["result"], "photo.jpg")
+        self.assertEqual(c.evaluate("$total")["body"]["result"], "900")
+        # no other round stops, and the total is the script's own
+        c.request("continue", {"threadId": 1})
+        self.assertEqual(c.wait_event("exited")["body"]["exitCode"], 3)
+        self.assertIn("3 files, 2800 bytes", c.output())
+
     def test_pipe_sourced_file_and_dollar_zero(self):
         c = self.client = Client()
         c.start(script("pipe.sh"), [(script("lib.sh"), 3), (script("pipe.sh"), 6)])