mvhutz/lambda
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add file input flag #12

Merged
mvhutz merged 2 commits from feat/file-input-flag into main 2026-01-11 21:43:12 +00:00
Conversation 0 Commits 2 Files Changed 3
+32 -13
mvhutz commented 2026-01-11 21:41:04 +00:00
Owner
Copy Link
Copy Source

Description

The lambda CLI previously only supported inline string expressions and stdin input.
This PR adds support for reading lambda expressions from files using the -f flag.
This makes it easier to work with larger programs stored in files.

Changes:

  • Added FileSource type to internal/config/source.go for reading from file paths.
  • Added -f flag to CLI argument parser with validation to prevent conflicting inputs.
  • Updated Makefile targets (run, profile, explain) to use -f flag instead of stdin redirection.

Decisions

The -f flag takes precedence over positional arguments.
If both are specified, an error is returned to avoid ambiguity.

Benefits

  • More intuitive workflow for file-based lambda programs.
  • Cleaner Makefile targets without stdin redirection.
  • Consistent with common CLI conventions (e.g., grep -f, awk -f).

Checklist

  • Code follows conventional commit format.
  • Branch follows naming convention (<type>/<description>).
  • Tests pass (if applicable).
  • Documentation updated (if applicable).
## Description The lambda CLI previously only supported inline string expressions and stdin input. This PR adds support for reading lambda expressions from files using the `-f` flag. This makes it easier to work with larger programs stored in files. Changes: - Added `FileSource` type to `internal/config/source.go` for reading from file paths. - Added `-f` flag to CLI argument parser with validation to prevent conflicting inputs. - Updated Makefile targets (`run`, `profile`, `explain`) to use `-f` flag instead of stdin redirection. ### Decisions The `-f` flag takes precedence over positional arguments. If both are specified, an error is returned to avoid ambiguity. ## Benefits - More intuitive workflow for file-based lambda programs. - Cleaner Makefile targets without stdin redirection. - Consistent with common CLI conventions (e.g., `grep -f`, `awk -f`). ## Checklist - [x] Code follows conventional commit format. - [x] Branch follows naming convention (`<type>/<description>`). - [ ] Tests pass (if applicable). - [ ] Documentation updated (if applicable).
mvhutz added 2 commits 2026-01-11 21:41:05 +00:00
feat: add file input flag to CLI. 82e1570c2c 
Added -f flag to allow reading lambda expressions from files.

Changes:
- Add FileSource type to read from file paths.
- Add -f flag to command-line parser.
- Implement validation to prevent conflicting -f and positional arguments.
refactor: use -f flag in Makefile targets. 9ab004bec4
mvhutz merged commit 62699a0e37 into main 2026-01-11 21:43:12 +00:00
mvhutz deleted branch feat/file-input-flag 2026-01-11 21:43:12 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
mvhutz (Maxim Hutz)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mvhutz/lambda#12