Merge pull request #10 from sonupreetam/014-skip-lockfile-only-pr

fix: skip automated PR when no doc content changes

Author: marcusburghardt

.github/workflows/sync-content-check.yml

@@ -23,18 +23,43 @@ jobs:
         with:
           go-version-file: go.mod
 
-      - name: Check for upstream changes
-        run: go run ./cmd/sync-content --org complytime --config sync-config.yaml --lock .content-lock.json --update-lock --summary sync-summary.md
+      - name: Update content lock
+        run: go run ./cmd/sync-content --org complytime --config sync-config.yaml --lock .content-lock.json --update-lock
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
+      - name: Check for doc content changes
+        id: check
+        run: go run ./cmd/sync-content --org complytime --config sync-config.yaml --lock .content-lock.json --write --summary sync-summary.md
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Derive PR title
+        id: title
+        if: steps.check.outputs.has_changes == 'true'
+        env:
+          CHANGED_REPOS: ${{ steps.check.outputs.changed_repos }}
+        run: |
+          IFS=',' read -ra repos <<< "$CHANGED_REPOS"
+          count=${#repos[@]}
+          if [ "$count" -le 3 ] && [ "$count" -gt 0 ]; then
+            names=$(printf "%s, " "${repos[@]}"); names="${names%, }"
+            title="content: sync ${names}"
+          else
+            repo_word="repositories"
+            [ "$count" -eq 1 ] && repo_word="repository"
+            title="content: sync ${count} ${repo_word}"
+          fi
+          echo "pr_title=${title}" >> "$GITHUB_OUTPUT"
+
       - name: Create or update PR
+        if: steps.check.outputs.has_changes == 'true'
         uses: peter-evans/create-pull-request@c0f553fe549906ede9cf27b5156039d195d2ece0
         with:
           add-paths: .content-lock.json
           branch: automated/content-sync-update
           commit-message: "content: update upstream documentation lockfile"
-          title: "content: update upstream documentation"
+          title: "${{ steps.title.outputs.pr_title }}"
           body-path: sync-summary.md
           labels: automated, documentation
           delete-branch: true

.gitignore

@@ -34,6 +34,20 @@ Thumbs.db
 *.swo
 *~
 
+# ─── AI tooling (tool-managed, installed locally via openspec init) ──
+# Run `openspec init --tools cursor` to restore these locally.
+.cursor/commands/
+.cursor/skills/
+.cursor/rules/specify-rules.mdc
+# OpenSpec working-change state — local only, analogous to .specify/scripts/.
+# openspec/config.yaml is tracked (project tool config).
+# Canonical specs live in specs/NNN-name/ (SpecKit format, org standard).
+openspec/changes/
+# SpecKit framework files (scripts and templates are tool-managed).
+.specify/scripts/
+.specify/templates/
+.specify/memory/
+
 # ─── Environment and secrets ─────────────────────────────────────────
 .env
 

cmd/sync-content/sync.go

@@ -90,8 +90,50 @@ func (r *syncResult) recordChangedRepoFile(repoName, srcPath string) {
 	r.mu.Unlock()
 }
 
+// changedRepos returns an alphabetically sorted, deduplicated list of repo
+// names that had content changes: repo-level adds/updates plus repos that had
+// individual file-level changes recorded in changedRepoFiles.
+func (r *syncResult) changedRepos() []string {
+	seen := make(map[string]struct{})
+	for _, name := range r.added {
+		seen[name] = struct{}{}
+	}
+	for _, name := range r.updated {
+		seen[name] = struct{}{}
+	}
+	for name, files := range r.changedRepoFiles {
+		if len(files) > 0 {
+			seen[name] = struct{}{}
+		}
+	}
+	repos := make([]string, 0, len(seen))
+	for name := range seen {
+		repos = append(repos, name)
+	}
+	sort.Strings(repos)
+	return repos
+}
+
+// changedFilesCount returns the total number of individual documentation files
+// that were written during this sync run across all repos.
+func (r *syncResult) changedFilesCount() int {
+	total := 0
+	for _, files := range r.changedRepoFiles {
+		total += len(files)
+	}
+	return total
+}
+
 func (r *syncResult) hasChanges() bool {
-	return len(r.added) > 0 || len(r.updated) > 0 || len(r.removed) > 0
+	if len(r.added) > 0 || len(r.updated) > 0 || len(r.removed) > 0 {
+		return true
+	}
+	for _, files := range r.changedRepoFiles {
+		if len(files) > 0 {
+			return true
+		}
+	}
+	return false
 }
 
 func shortSHA(sha string) string {
@@ -153,6 +195,18 @@ func (r *syncResult) toMarkdown() string {
 	sort.Strings(removed)
 	sort.Strings(unchanged)
 
+	if repos := r.changedRepos(); len(repos) > 0 {
+		repoWord := "repositories"
+		if len(repos) == 1 {
+			repoWord = "repository"
+		}
+		names := make([]string, len(repos))
+		for i, name := range repos {
+			names[i] = "`" + name + "`"
+		}
+		fmt.Fprintf(&b, "**Changed %s**: %s\n\n", repoWord, strings.Join(names, ", "))
+	}
+
 	if len(added) > 0 {
 		b.WriteString("### New Repositories\n\n")
 		for _, name := range added {
@@ -737,6 +791,8 @@ func writeGitHubOutputs(result *syncResult) {
 			}
 			_, _ = fmt.Fprintf(f, "has_changes=%s\n", hasChanges)
 			_, _ = fmt.Fprintf(f, "changed_count=%d\n", len(result.added)+len(result.updated))
+			_, _ = fmt.Fprintf(f, "changed_repos=%s\n", strings.Join(result.changedRepos(), ","))
+			_, _ = fmt.Fprintf(f, "changed_files_count=%d\n", result.changedFilesCount())
 			_, _ = fmt.Fprintf(f, "files_processed=%d\n", result.filesProcessed)
 			_, _ = fmt.Fprintf(f, "error_count=%d\n", result.errors)
 		}

cmd/sync-content/sync_test.go

@@ -996,6 +996,67 @@ func TestToMarkdown_NoChanges(t *testing.T) {
 	}
 }
 
+func TestHasChanges(t *testing.T) {
+	tests := []struct {
+		name   string
+		result syncResult
+		want   bool
+	}{
+		{
+			name:   "empty result",
+			result: syncResult{},
+			want:   false,
+		},
+		{
+			name:   "only unchanged repos",
+			result: syncResult{unchanged: []string{"repo-a"}},
+			want:   false,
+		},
+		{
+			name:   "added repo",
+			result: syncResult{added: []string{"new-repo"}},
+			want:   true,
+		},
+		{
+			name:   "updated repo",
+			result: syncResult{updated: []string{"existing-repo"}},
+			want:   true,
+		},
+		{
+			name:   "removed repo",
+			result: syncResult{removed: []string{"old-repo"}},
+			want:   true,
+		},
+		{
+			name: "doc file changed without repo-level change",
+			result: syncResult{
+				unchanged: []string{"some-repo"},
+				changedRepoFiles: map[string][]string{
+					"some-repo": {"docs/guide.md"},
+				},
+			},
+			want: true,
+		},
+		{
+			name: "changedRepoFiles present but empty slice",
+			result: syncResult{
+				changedRepoFiles: map[string][]string{
+					"some-repo": {},
+				},
+			},
+			want: false,
+		},
+	}
+
+	for i := range tests {
+		t.Run(tests[i].name, func(t *testing.T) {
+			if got := tests[i].result.hasChanges(); got != tests[i].want {
+				t.Errorf("hasChanges() = %v, want %v", got, tests[i].want)
+			}
+		})
+	}
+}
+
 func TestToMarkdown_FallbackWithoutDetails(t *testing.T) {
 	result := &syncResult{
 		synced: 1,
@@ -1357,3 +1418,124 @@ func TestParseNameList_RepoFilterOverridesExclude(t *testing.T) {
 		t.Error("complyctl should be in includeSet")
 	}
 }
+
+func TestChangedRepos(t *testing.T) {
+	tests := []struct {
+		name   string
+		result syncResult
+		want   []string
+	}{
+		{
+			name:   "empty result",
+			result: syncResult{},
+			want:   []string{},
+		},
+		{
+			name:   "repo-level added only",
+			result: syncResult{added: []string{"repo-b", "repo-a"}},
+			want:   []string{"repo-a", "repo-b"},
+		},
+		{
+			name:   "repo-level updated only",
+			result: syncResult{updated: []string{"repo-c"}},
+			want:   []string{"repo-c"},
+		},
+		{
+			name: "file-level changes only (no repo-level add/update)",
+			result: syncResult{
+				unchanged: []string{"repo-x"},
+				changedRepoFiles: map[string][]string{
+					"repo-x": {"docs/guide.md"},
+				},
+			},
+			want: []string{"repo-x"},
+		},
+		{
+			name: "overlap: repo in both updated and changedRepoFiles",
+			result: syncResult{
+				updated: []string{"repo-z"},
+				changedRepoFiles: map[string][]string{
+					"repo-z": {"docs/page.md"},
+					"repo-a": {"docs/intro.md"},
+				},
+			},
+			want: []string{"repo-a", "repo-z"},
+		},
+		{
+			name: "changedRepoFiles with empty slice excluded",
+			result: syncResult{
+				changedRepoFiles: map[string][]string{
+					"repo-empty": {},
+				},
+			},
+			want: []string{},
+		},
+	}
+
+	for i := range tests {
+		t.Run(tests[i].name, func(t *testing.T) {
+			got := tests[i].result.changedRepos()
+			if got == nil {
+				got = []string{}
+			}
+			if len(got) != len(tests[i].want) {
+				t.Fatalf("changedRepos() = %v, want %v", got, tests[i].want)
+			}
+			for j := range got {
+				if got[j] != tests[i].want[j] {
+					t.Errorf("changedRepos()[%d] = %q, want %q", j, got[j], tests[i].want[j])
+				}
+			}
+		})
+	}
+}
+
+func TestChangedFilesCount(t *testing.T) {
+	tests := []struct {
+		name   string
+		result syncResult
+		want   int
+	}{
+		{
+			name:   "zero files",
+			result: syncResult{},
+			want:   0,
+		},
+		{
+			name: "single repo with files",
+			result: syncResult{
+				changedRepoFiles: map[string][]string{
+					"repo-a": {"docs/a.md", "docs/b.md"},
+				},
+			},
+			want: 2,
+		},
+		{
+			name: "multiple repos",
+			result: syncResult{
+				changedRepoFiles: map[string][]string{
+					"repo-a": {"docs/a.md"},
+					"repo-b": {"docs/b.md", "docs/c.md", "docs/d.md"},
+				},
+			},
+			want: 4,
+		},
+		{
+			name: "empty slice counts as zero",
+			result: syncResult{
+				changedRepoFiles: map[string][]string{
+					"repo-a": {},
+				},
+			},
+			want: 0,
+		},
+	}
+
+	for i := range tests {
+		t.Run(tests[i].name, func(t *testing.T) {
+			if got := tests[i].result.changedFilesCount(); got != tests[i].want {
+				t.Errorf("changedFilesCount() = %d, want %d", got, tests[i].want)
+			}
+		})
+	}
+}

openspec/config.yaml

@@ -0,0 +1,20 @@
+schema: spec-driven
+
+# Project context (optional)
+# This is shown to AI when creating artifacts.
+# Add your tech stack, conventions, style guides, domain knowledge, etc.
+# Example:
+#   context: |
+#     Tech stack: TypeScript, React, Node.js
+#     We use conventional commits
+#     Domain: e-commerce platform
+
+# Per-artifact rules (optional)
+# Add custom rules for specific artifacts.
+# Example:
+#   rules:
+#     proposal:
+#       - Keep proposals under 500 words
+#       - Always include a "Non-goals" section
+#     tasks:
+#       - Break tasks into chunks of max 2 hours