fix: handle spaces in file paths in Modelfile parsing

Fixes issue #207 where Modelfile parsing would fail with 'invalid args' error
when file paths contained spaces without quotes.

Changes:
- Modified parseStringArgs() to join multiple arguments with spaces
- This handles unquoted paths like 'CONFIG example workflows/file.json'
- Maintains backward compatibility with quoted paths
- Updated tests to reflect new behavior
- Added comprehensive test coverage for edge cases

The fix allows both:
- CONFIG "path with spaces/file.json" (quoted - existing behavior)
- CONFIG path with spaces/file.json (unquoted - new behavior)

Signed-off-by: Ricardo Aravena <[email protected]>

Author: raravena80

pkg/modelfile/modelfile_test.go

@@ -1308,8 +1308,8 @@ MODEL model weights.bin
 CODE inference script.py
 DOC README file.md
 `,
-			expectError: true,
-			description: "Unquoted paths with spaces should cause parsing errors",
+			expectError: false,
+			description: "Unquoted paths with spaces are now handled correctly by joining arguments",
 		},
 		{
 			name: "quoted_paths_with_spaces",

pkg/modelfile/parser/args_parser.go

@@ -18,18 +18,24 @@ package parser
 
 import (
 	"errors"
+	"strings"
 )
 
 // parseStringArgs parses the string type of args and returns a Node, for example:
 // "MODEL foo" args' value is "foo".
+// If multiple args are provided (due to unquoted spaces), they are joined with spaces.
+// This handles cases like: CONFIG path with spaces/file.json
 func parseStringArgs(args []string, start, end int) (Node, error) {
-	if len(args) != 1 {
-		return nil, errors.New("invalid args")
+	if len(args) == 0 {
+		return nil, errors.New("empty args")
 	}
 
-	if args[0] == "" {
+	// Join all arguments with spaces to handle unquoted file paths with spaces
+	joined := strings.Join(args, " ")
+
+	if joined == "" {
 		return nil, errors.New("empty args")
 	}
 
-	return NewNode(args[0], start, end), nil
+	return NewNode(joined, start, end), nil
 }

pkg/modelfile/parser/args_parser_test.go

@@ -33,8 +33,11 @@ func TestParseStringArgs(t *testing.T) {
 		{[]string{"foo"}, 1, 2, false, "foo"},
 		{[]string{"bar"}, 3, 4, false, "bar"},
 		{[]string{}, 5, 6, true, ""},
-		{[]string{"foo", "bar"}, 7, 8, true, ""},
+		{[]string{"foo", "bar"}, 7, 8, false, "foo bar"}, // Now handles multiple args by joining
 		{[]string{""}, 9, 10, true, ""},
+		// Additional test cases for spaces in file paths
+		{[]string{"path", "with", "spaces/file.json"}, 11, 12, false, "path with spaces/file.json"},
+		{[]string{"example", "workflows_Wan2.1/image_to_video_wan_480p_example.json"}, 13, 14, false, "example workflows_Wan2.1/image_to_video_wan_480p_example.json"},
 	}
 
 	assert := assert.New(t)