Add typed causes for a rejected source and for an opaque argument value
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start by reading the parser’s command-name rejection path and the argument analysis types that expose Unknown with ArgKind.DynamicSkip; the issue does not name specific files or tests. Trace how ShellOpaqueCause is currently represented and exposed. Done means parse failures carry a typed reason and argument values expose the three listed opaque causes as public typed facts.
由索引模型根据 Issue 内容生成。
描述
Problem
Since 0.4.0-beta.18, the parser rejects a source with a brace list in the
command name, such as {"b":2,"nested":{"c":3}}. That is correct, because
Bash expands the word to another program and its operands. But the parser
gives the consumer only an error string. Also, a brace word in an argument
position reaches the consumer as an Unknown value with ArgKind.DynamicSkip,
the same as an arithmetic word or a command substitution. The internal
ShellOpaqueCause is not public.
Netclaw gives the agent a rewrite correction for some unresolved forms. It
needs the cause to choose the correct advice:
- A rejected source with a brace list in the command name: before beta.18,
the agent got the "run each command separately" correction. Now 30 corpus
rows of JSON text get a prompt, or an unattended denial. Netclaw must not
match the error string. git {push,fetch} origin: Netclaw now gives the general "write the command
words literally" advice, because it cannot see that the word is a brace list.
Request
- Add a typed reason to a parse failure, for example an enum value
BraceExpansionInCommandName, next to the error text. - Expose the opaque cause of an argument value as a public typed fact, for
exampleAnalyzedArgument.OpaqueCauseswith the valuesBraceExpansion,
ArithmeticExpansion, andCommandSubstitution.
Both facts are general shell syntax facts. They do not depend on a program.
Consumer
netclaw-dev/netclaw#2349, review comment 4187346461.
- 主要语言
- C#
- 星标
- 15
- 派生
- 0
- 平均合并
- 9 小时 54 分钟
- 30 天内合并 PR
- 40
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Aaronontheweb/ShellSyntaxTree 的其他 Issue
-
Command words: restore the beta.19 verb-slot rule可能已有人在做 @Aaronontheweb 今天认领。 未关闭
难度 3/5 1-2 天 新手友好度 35/100
Aaronontheweb/ShellSyntaxTree#240 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 55/100
Aaronontheweb/ShellSyntaxTree#235 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 38/100
Aaronontheweb/ShellSyntaxTree#227 · 1 条评论 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 35/100
Aaronontheweb/ShellSyntaxTree#190 ·
维护者通常 1 天内回复
-
enhancement
难度 5/5 一周以上 新手友好度 35/100
Aaronontheweb/ShellSyntaxTree#71 · 1 条评论 ·
维护者通常 1 天内回复
查看 Aaronontheweb/ShellSyntaxTree 的全部 Issue
相似的 Issue
-
S: Untriaged
难度 2/5 1-3 小时 新手友好度 78/100
space-wizards/space-station-14#46357 ·
维护者通常 1 天内回复
-
Versioning_oM: Remove unneeded depeendecy on Test_oM可能已有人在做 @IsakNaslundBh 今天认领。 未关闭type:bug type:compliance
难度 2/5 1 小时以内 新手友好度 85/100
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 66/100
MicrosoftLearning/PL-400_Microsoft-Power-Platform-Developer#231 ·
-
难度 2/5 1-3 小时 新手友好度 66/100
joinrpg/joinrpg-net#5313 ·
维护者通常 1 天内回复