From 78054a53525901c2a36b1bb0f9ae4ba6716572bc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20=C5=BD=C3=A1dn=C3=ADk?= <kubouch@gmail.com>
Date: Sun, 12 Sep 2021 15:29:27 +0300
Subject: [PATCH 1/3] Allow parsing left-unbounded range (..10)

It is implemented as a preliminary check when parsing a call and relies
on a fact that a token that successfully parses as a range is unlikely
to be a valid path or command name.
---
 crates/nu-parser/src/parser.rs        |  9 ++++++++
 crates/nu-parser/tests/test_parser.rs | 31 +++++++++++++++++++++++++++
 2 files changed, 40 insertions(+)

diff --git a/crates/nu-parser/src/parser.rs b/crates/nu-parser/src/parser.rs
index 6199e87ee..839f43db8 100644
--- a/crates/nu-parser/src/parser.rs
+++ b/crates/nu-parser/src/parser.rs
@@ -469,6 +469,15 @@ pub fn parse_call(
     spans: &[Span],
     expand_aliases: bool,
 ) -> (Expression, Option<ParseError>) {
+    // We might be parsing left-unbounded range ("..10")
+    let bytes = working_set.get_span_contents(spans[0]);
+    if let (Some(b'.'), Some(b'.')) = (bytes.get(0), bytes.get(1)) {
+        let (range_expr, range_err) = parse_range(working_set, spans[0]);
+        if range_err.is_none() {
+            return (range_expr, range_err);
+        }
+    }
+
     // assume spans.len() > 0?
     let mut pos = 0;
     let mut shorthand = vec![];
diff --git a/crates/nu-parser/tests/test_parser.rs b/crates/nu-parser/tests/test_parser.rs
index 2015f356c..11c8189f7 100644
--- a/crates/nu-parser/tests/test_parser.rs
+++ b/crates/nu-parser/tests/test_parser.rs
@@ -371,6 +371,37 @@ mod range {
         }
     }
 
+    #[test]
+    fn parse_left_unbounded_range() {
+        let engine_state = EngineState::new();
+        let mut working_set = StateWorkingSet::new(&engine_state);
+
+        let (block, err) = parse(&mut working_set, None, b"..10", true);
+
+        assert!(err.is_none());
+        assert!(block.len() == 1);
+        match &block[0] {
+            Statement::Pipeline(Pipeline { expressions }) => {
+                assert!(expressions.len() == 1);
+                assert!(matches!(
+                    expressions[0],
+                    Expression {
+                        expr: Expr::Range(
+                            None,
+                            Some(_),
+                            RangeOperator {
+                                inclusion: RangeInclusion::Inclusive,
+                                ..
+                            }
+                        ),
+                        ..
+                    }
+                ))
+            }
+            _ => panic!("No match"),
+        }
+    }
+
     #[test]
     fn parse_negative_range() {
         let engine_state = EngineState::new();

From 8577d3ff413e6cecf8f84e03f89080c99995632f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20=C5=BD=C3=A1dn=C3=ADk?= <kubouch@gmail.com>
Date: Sun, 12 Sep 2021 15:48:19 +0300
Subject: [PATCH 2/3] Check for left-unbounded range before external cmd

---
 crates/nu-parser/src/parser.rs | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/crates/nu-parser/src/parser.rs b/crates/nu-parser/src/parser.rs
index 839f43db8..27acdeca6 100644
--- a/crates/nu-parser/src/parser.rs
+++ b/crates/nu-parser/src/parser.rs
@@ -469,15 +469,6 @@ pub fn parse_call(
     spans: &[Span],
     expand_aliases: bool,
 ) -> (Expression, Option<ParseError>) {
-    // We might be parsing left-unbounded range ("..10")
-    let bytes = working_set.get_span_contents(spans[0]);
-    if let (Some(b'.'), Some(b'.')) = (bytes.get(0), bytes.get(1)) {
-        let (range_expr, range_err) = parse_range(working_set, spans[0]);
-        if range_err.is_none() {
-            return (range_expr, range_err);
-        }
-    }
-
     // assume spans.len() > 0?
     let mut pos = 0;
     let mut shorthand = vec![];
@@ -601,6 +592,14 @@ pub fn parse_call(
             err,
         )
     } else {
+        // We might be parsing left-unbounded range ("..10")
+        let bytes = working_set.get_span_contents(spans[0]);
+        if let (Some(b'.'), Some(b'.')) = (bytes.get(0), bytes.get(1)) {
+            let (range_expr, range_err) = parse_range(working_set, spans[0]);
+            if range_err.is_none() {
+                return (range_expr, range_err);
+            }
+        }
         parse_external_call(working_set, spans)
     }
 }

From e6a2e27e3360be80cd4e86f6bccc1fdade1db922 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20=C5=BD=C3=A1dn=C3=ADk?= <kubouch@gmail.com>
Date: Sun, 12 Sep 2021 15:57:49 +0300
Subject: [PATCH 3/3] Fix failing compilation after rebase

---
 crates/nu-parser/tests/test_parser.rs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/crates/nu-parser/tests/test_parser.rs b/crates/nu-parser/tests/test_parser.rs
index 11c8189f7..ca5f3d13c 100644
--- a/crates/nu-parser/tests/test_parser.rs
+++ b/crates/nu-parser/tests/test_parser.rs
@@ -387,6 +387,7 @@ mod range {
                     expressions[0],
                     Expression {
                         expr: Expr::Range(
+                            None,
                             None,
                             Some(_),
                             RangeOperator {