Add typed causes for a rejected source and for an opaque argument value
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
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時間 8分
- マージ済み PR（30日）
- 46
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Aaronontheweb/ShellSyntaxTree のほかの issue
-
Bash: an assignment in a loop body that reads the loop variable is not listed on later commandsオープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
Aaronontheweb/ShellSyntaxTree#249 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 57/100
Aaronontheweb/ShellSyntaxTree#244 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
Aaronontheweb/ShellSyntaxTree#235 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
Aaronontheweb/ShellSyntaxTree#227 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
Aaronontheweb/ShellSyntaxTree#190 ·
メンテナーはふだん 1 日以内に返信
Aaronontheweb/ShellSyntaxTree の issue をすべて見る
似ている issue
-
Bug pulumi/pulumi
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
activescott/lessmsi#306 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
HTML sitemap lists unpublished pages対応中かも @KrzysztofPajak が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
grandnode/grandnode2#883 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信