Add a fact for an unquoted glob character in the authored word
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start by locating AnalyzedArgument.MayPathnameExpand and the argument analysis code that sets it; the issue does not name files or tests. Trace how authored words and quoting are represented, then add a separate fact for unquoted authored glob characters while preserving MayPathnameExpand semantics. Verify the examples in the issue, including quoted and expansion-only cases; done means the new fact distinguishes them without changing the existing fact.
索引モデルが issue の本文から書いたものです。
説明
Problem
AnalyzedArgument.MayPathnameExpand (0.4.0-beta.19) is true for two different
cases:
- The authored word has an unquoted glob character (
*,?,[), for
example../netclaw/"${d}s"/*. - The authored word has an unquoted expansion whose value can contain a glob
character, for example$naftern=$(date).
A consumer cannot tell these cases apart. Netclaw needs the difference for an
output command (echo, printf):
- In case 1, the authored glob can list protected file names, so the consumer
applies its glob rule. - In case 2, the owner decision of netclaw-dev/netclaw#2349 keeps the operand
as data.
Today Netclaw cannot apply that decision when an assignment supplies the
value. n=$(date); echo $n gets one exact prompt, and an unattended run
denies it. An attempt to treat every such operand as data also made
d=key; echo ../netclaw/"${d}s"/* data, which is not safe.
Request
Add a general per-argument fact that is true only when the authored word has
an unquoted glob character. A suggested name is HasUnquotedGlobCharacter.
MayPathnameExpand keeps its meaning.
Examples
| Word | MayPathnameExpand |
New fact |
|---|---|---|
$n |
true | false |
/work/$f |
true | false |
"${d}s"/* |
true | true |
*.cs |
true | true |
"$n" |
false | false |
'*.cs' |
false | false |
A brace word stays opaque and needs no new value.
Consumer
netclaw-dev/netclaw#2349, review comment 4187346452.
- 主要言語
- 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 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
Aaronontheweb/ShellSyntaxTree#236 ·
メンテナーはふだん 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
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
activescott/lessmsi#306 ·
-
Python: Bug: split_plaintext_paragraph / split_markdown_paragraph can return a chunk larger than max_tokens対応中かも @xThreeh が今日担当しました。 オープンpython triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
microsoft/semantic-kernel#14566 ·
メンテナーはふだん 4 日以内に返信
-
triage
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
rjmurillo/moq.analyzers#1384 ·
-
Variables passed to Compensated are not set on the routing slip再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
MassTransit/MassTransit#6249 ·
-
security
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Sendspin/sendspin-dotnet#339 ·
メンテナーはふだん 1 日以内に返信