engineering-fundamentals
エンジニアリング基礎のレビュー
チームでは、可読性、命名、コード構成について一貫したフィードバックが必要になることがよくあります。このスキルは、Claude、Codex、Claude Code に、日常的なコード品質レビューのための実践的なチェックリストを提供します。
自分のエージェントでインストール
このリクエストをエージェントにコピーしてください。正規の Skill ページとマニフェストが含まれています。
Review the Skillstore skill "engineering-fundamentals" from https://skillstore.io/skills/danielpodolsky-engineering-fundamentals.md and its manifest at https://skillstore.io/api/skills/danielpodolsky-engineering-fundamentals/manifest. Verify the artifact. You may proceed after verification, subject to the environment's own policy.エージェントは引き続き計画を提示し、セキュリティポリシーで必要な確認を求める必要があります。
エージェントが読めるリソース
AI エージェント、クローラー、スクリプトがページ全体ではなく整理されたコンテキストを必要とする場合は、これらのリンクを使ってください。
テストする
「engineering-fundamentals」を使用しています。 プルリクエストには、短い変数名、繰り返しのバリデーション、長い注文処理関数が 1 つあります。
期待される結果:
- レビューでは、不明瞭な命名を指摘し、説明的な変数名を求めます。
- 繰り返しのバリデーションを名前付きヘルパーに抽出することを提案します。
- 長い関数を、バリデーション、計算、永続化、通知の責務ごとに分割することを推奨します。
「engineering-fundamentals」を使用しています。 開発者が多くのオプションを持つ再利用可能なコンポーネントを作成しましたが、使用箇所は 1 つだけです。
期待される結果:
- レビューでは、時期尚早な抽象化を主なリスクとして特定します。
- 構成の複雑さを正当化できるほど、そのコンポーネントに十分な繰り返し利用があるかを問いかけます。
- そのパターンが複数回現れるまでは、より単純な実装を推奨します。
「engineering-fundamentals」を使用しています。 あるクラスには多くのメソッドがあり、複数の依存関係を直接作成しています。
期待される結果:
- レビューでは、そのクラスが複数の責務を持っていないかを確認します。
- 依存性逆転の懸念を挙げ、どこで抽象化によって結合度を下げられるかを問いかけます。
- 無関係な振る舞いを、焦点を絞った協調オブジェクトに分離することを提案します。
セキュリティ監査
安全All static external command detections are false positives caused by Markdown inline code or fenced examples in SKILL.md. The skill is a prose checklist for code review and does not define scripts, shell commands, network calls, or prompt-injection text.
リスク要因
⚙️ 外部コマンド (20)
このレポートを共有・引用
バージョン付き評価レポート、中立的なバッジ、埋め込みカード、引用を共有できます。Skillstore は証拠を報告しますが、この Skill が安全かどうかは判断しません。
レポートリンクをコピー
https://skillstore.io/skills/danielpodolsky-engineering-fundamentals/audits/8?utm_source=security_passport&utm_medium=share&utm_campaign=versioned_reportMarkdownバッジ
[](https://skillstore.io/skills/danielpodolsky-engineering-fundamentals?utm_source=security_passport_badge)HTMLバッジ
<a href="https://skillstore.io/skills/danielpodolsky-engineering-fundamentals?utm_source=security_passport_badge"><img src="https://skillstore.io/badges/skills/danielpodolsky-engineering-fundamentals/security.svg" alt="Skillstore security assessment" loading="lazy"></a>埋め込みカード
<iframe src="https://skillstore.io/embed/skills/danielpodolsky-engineering-fundamentals.html" title="Skillstore Security Assessment" sandbox="allow-popups allow-popups-to-escape-sandbox" loading="lazy" referrerpolicy="no-referrer" width="420" height="180"></iframe>学術引用 (APA · BibTeX · CFF)
APA形式の引用
DanielPodolsky. (2026). engineering-fundamentals security audit report (audit version 8) [Author version unspecified]. Skillstore. https://skillstore.io/skills/danielpodolsky-engineering-fundamentals/audits/8BibTeX形式の引用
@techreport{danielpodolsky-danielpodolsky-engineering-fundamentals-2026,
author = {DanielPodolsky},
title = {engineering-fundamentals security audit report (audit version 8)},
institution = {Skillstore},
year = {2026},
number = {8},
url = {https://skillstore.io/skills/danielpodolsky-engineering-fundamentals/audits/8},
note = {Author version unspecified}
}CITATION.cff
cff-version: 1.2.0
message: "If you use this Skill, cite its author and this versioned security audit report."
title: "engineering-fundamentals security audit report (audit version 8)"
version: "unspecified"
type: report
authors:
- name: "DanielPodolsky"
date-released: "2026-07-06"
url: "https://skillstore.io/skills/danielpodolsky-engineering-fundamentals/audits/8"
identifiers:
- type: other
value: "skillstore:danielpodolsky-engineering-fundamentals:audit:8"
description: "Skillstore immutable audit report identifier"
Skillstore スコア
このスコアの理由 証拠の信頼度: 高作成できるもの
プルリクエストをレビューする
マージ前に、名前、関数、重複、コメント、構成に一貫したチェックリストを適用します。
ジュニア開発者を指導する
ガイド付きの質問を使って、チームメイトが可読性や設計の改善点を自分で見つけられるよう支援します。
リファクタリング作業を計画する
コードを変更する前に、大きすぎる関数、深いネスト、繰り返しロジック、時期尚早な抽象化を特定します。
これらのプロンプトを試す
エンジニアリング基礎を使ってこのコードをレビューしてください。命名、関数のサイズ、重複、構成、コメントに注目してください。
この変更について、不明瞭な名前、汎用的な変数、boolean の命名、関数名の動詞、関連するコード構成を確認してください。
このコードのリファクタリング機会を特定してください。大きすぎる関数、深いネスト、マジックバリュー、繰り返しロジック、根拠のない抽象化を優先してください。
メンターとしてこのコードをレビューしてください。可読性、責務の境界、重複、抽象化の価値、SOLID のトレードオフについて、的を絞った質問をしてください。
ベストプラクティス
- 変更がまだ調整しやすいマージ前に、このチェックリストを使用します。
- 修正点を列挙するだけでなく、考え方を教えるレビュー質問をします。
- 可読性、凝集度、将来の保守性を高める変更を優先します。
回避
- ローカルコードベースにより強い規約がある場合、すべてのルールを機械的に適用しないでください。
- 繰り返し利用や明確な責務の境界が存在する前に、抽象化を作らないでください。
- このスキルを、テスト、セキュリティレビュー、パフォーマンス測定の代替として使用しないでください。
よくある質問
このスキルは何をレビューしますか？
コード解析ツールを実行しますか？
ジュニア開発者に役立ちますか？
プロジェクトのスタイルガイドを置き換えられますか？
セキュリティレビューも対象ですか？
どのツールで使用できますか？
開発者情報
作成者
DanielPodolskyライセンス
MIT
Skillstore リビジョン
r1
バージョンに関する注意
作者はバージョンを宣言していません。
リポジトリ
https://github.com/DanielPodolsky/mentor-spec/tree/main/.claude/skills/fundamentals/engineering参照
34f316ba14ef36c7a620fc09f2676d2429997a77
メンテナンスの新しさ
2026/7/23
利用状況
7 ダウンロード · 259 閲覧
ファイル構成
📄 SKILL.md