list links from specified page

[JQinglong]

InstallActionに対して、-P でページを指定して、-eでリンクURLの絞り込み条件を指定するようにしました。
現時点では抽出したurlをconsoleに出力しています。
このようにして複数取得したurlについて実際のInstallAction（その他のオプション指定に従ってコンテンツの取得）を行いたいですが、そのためには、InstallActionの実体の処理を切り出して、それを呼び出すようにする必要があるのではないかと思いましたが、既存ソースへの改変が大きいため相談させていただきました。

[ryo-ma]

InstallActionの実体の処理を切り出して
どちらの部分のコードの想定でしょうか？

私の認識では以下の部分installFromURLをループで実行できれば良いと思っていますがいかがでしょうか？

dim/libs/actions.ts

Lines 82 to 97 in 337de17

	const fullPath = await installFromURL(
	url,
	options.name,
	options.postProcesses,
	parsedHeaders,
	).catch(
	(error) => {
	console.error(
	Colors.red("Failed to install."),
	Colors.red(error.message),
	);
	Deno.exit(1);
	},
	);
	console.log(
	Colors.green(`Installed to ${fullPath}`),

また、parsedHeadersも利用したいため、今回実装しているpageInstalll の処理をparsedHeadersの宣言以降に移動させたほうがいいですね。
最終的には、今回実装のpageInstallに関してはaction_helper/installer.tsの方にまとめてしまいたいと思ってます。
でも、まずは一旦は動く様にしましょうか。

[ryo-ma]

下部のCIで落ちてしまっているのでdeno fmtコマンドによってフォーマットしていただければと思います。

[ryo-ma]

CIがまたもやdeno fmtで落ちてますね。

[ryo-ma]

一通り修正が完了したら今回の処理部分の try の中の処理を 他のインストール処理と同様installFromPage()のような感じで libs/action_helper/installer.tsに処理をまとめたいですね。

[ryo-ma]

Github Action(CI)のlintでひっかかっちゃいましたね。 deno lint でチェックすると出てきます。

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7196131) 100.00% compared to head (08ce138) 100.00%.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #107   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines         1429      1510   +81     
  Branches       147       156    +9     
=========================================
+ Hits          1429      1510   +81     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ryo-ma]

また少しフォーマットでCIが落ちてしまってますね。

[ryo-ma]

@JQinglong
あと以下の行だけですね。ここの分岐を通せたりしますか？それができて、あらためて動作確認したらマージしたいと思います。

https://app.codecov.io/gh/c-3lab/dim/pull/107?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=c-3lab#4663a8fc79877a2f7b91c1d0c139accd-R322

[JQinglong]

@ryo-ma
未実行になっている行ですが、何が問題でこれが必要だったのかわからなくなってしまったので、一旦外しました。
また前段階で何かのチェックにかかってしまうかもしれませんが、すみません・・・

[ryo-ma]

@JQinglong ありがとうございます。エラー処理周りは少し気になるところもありますが、とりあえずマージしてしまいましょうか。使っている中で問題があればまたプルリクで対応してきましょう。

[JQinglong]

マージありがとうございました。ぜひ、自分の開発でも使って、フィードバックもしていきたいと思っています。