Setup presubmit for dart_skills_lint

.github/workflows/dart_skills_lint_workflow.yaml

@@ -0,0 +1,65 @@
+name: dart_skills_lint
+permissions: read-all
+
+on:
+  pull_request:
+    paths:
+      - 'skills/**'
+      - 'tool/dart_skills_lint/**'
+      - '.github/workflows/dart_skills_lint_workflow.yaml'
+  push:
+    branches: [ main ]
+    paths:
+      - 'skills/**'
+      - 'tool/dart_skills_lint/**'
+      - '.github/workflows/dart_skills_lint_workflow.yaml'
+  schedule:
+    - cron: '0 0 * * 0' # weekly
+
+defaults:
+  run:
+    working-directory: tool/dart_skills_lint
+
+jobs:
+  analyze_and_test:
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v6
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: stable
+
+      - run: dart pub get
+
+      - run: dart analyze --fatal-infos
+
+      - run: dart test
+
+  validate_skills:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: stable
+
+      - run: dart pub get
+
+      - name: Validate skills
+        run: dart run dart_skills_lint
+
+  formatting:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: stable
+
+      - run: dart pub get
+
+      - run: dart format --output=none --set-exit-if-changed .

tool/dart_skills_lint/analysis_options.yaml

@@ -9,9 +9,6 @@
 #
 #   - analysis_options.yaml (this file)
 #   - https://github.com/flutter/packages/blob/main/analysis_options.yaml
-#
-# This file contains the analysis options used for code in the flutter/flutter
-# repository.
 
 analyzer:
   language:

tool/dart_skills_lint/dart_skills_lint.yaml

@@ -1,6 +1,7 @@
 dart_skills_lint:
   rules:
     check-relative-paths: error
+    check-absolute-paths: error
   directories:
     - path: ".agents/skills"
       ignore_file: ".agents/skills/ignore.json"

tool/dart_skills_lint/lib/src/validator.dart

@@ -55,8 +55,6 @@ class Validator {
     return _ruleOverrides[rule.name] ?? rule.defaultSeverity;
   }
 
-
-
   static const _nameField = 'name';
   static const _descriptionField = 'description';
   static const _licenseField = 'license';

tool/dart_skills_lint/test/absolute_paths_test.dart

@@ -35,8 +35,8 @@ void main() {
       final ValidationResult result = await validator.validate(skillDir);
 
       expect(result.isValid, isTrue);
-      expect(
-          result.warnings, contains(contains('Absolute filepath found in link: /absolute/path.md')));
+      expect(result.warnings,
+          contains(contains('Absolute filepath found in link: /absolute/path.md')));
     });
 
     test('flags windows absolute path starting with drive letter as error', () async {

tool/dart_skills_lint/test/resolve_rules_test.dart

@@ -23,7 +23,7 @@ void main() {
     test('returns defaults when no args and empty config', () {
       final ArgResults results = createParser().parse([]);
       final config = Configuration();
-      
+
       final Map<String, AnalysisSeverity> resolved = resolveRules(results, config);
 
       expect(resolved[relativePathsCheck.name], relativePathsCheck.defaultSeverity);
@@ -41,7 +41,7 @@ void main() {
         relativePathsCheck.name: AnalysisSeverity.error,
         absolutePathsCheck.name: AnalysisSeverity.warning,
       });
-      
+
       final Map<String, AnalysisSeverity> resolved = resolveRules(results, config);
 
       expect(resolved[relativePathsCheck.name], AnalysisSeverity.error);
@@ -55,7 +55,7 @@ void main() {
       final config = Configuration(configuredRules: {
         relativePathsCheck.name: AnalysisSeverity.error,
       });
-      
+
       final Map<String, AnalysisSeverity> resolved = resolveRules(results, config);
 
       expect(resolved[relativePathsCheck.name], AnalysisSeverity.warning);
@@ -66,7 +66,7 @@ void main() {
       final config = Configuration(configuredRules: {
         validYamlMetadataCheck.name: AnalysisSeverity.warning,
       });
-      
+
       final Map<String, AnalysisSeverity> resolved = resolveRules(results, config);
 
       expect(resolved[validYamlMetadataCheck.name], AnalysisSeverity.disabled);