docs: add Claude framework roadmap and fix workflow paths#116
Open
bgauduch wants to merge 7 commits into
Open
Conversation
Freeze the plan agreed during the planning session as a source of truth for the agentic tooling adoption. Survives context loss and serves as onboarding reference for future contributors. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
The workflow watched "hashicorp.asc" at repo root and "supported_platforms.json" which doesn't exist. Replace with "security/**" and drop the dead entry so the trigger reflects reality. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
The workflow watched "hashicorp.asc" at repo root, but the file lives under security/. Replace with "security/**" so the trigger fires on actual changes. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
…pdate The paths filter referenced "dockerhub-description.yml" but the actual file is "dockerhub-description-update.yml", so the workflow never triggered on its own changes. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
The guide referenced Debian bullseye and listed only Git and JQ, while the Dockerfile now uses bookworm and installs ca-certificates, git, jq, openssh-client in the final stage plus curl, gnupg, unzip in build stages. Also clarifies that security files live under security/ (e.g. security/hashicorp.asc). https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
Capture the development conventions binding any Claude session under this framework. Most critically: never merge a PR, push to master, create a PR, or force-push without explicit human approval. Will be extracted to a dedicated conventions doc in Phase 1 and reflected in CLAUDE.md in Phase 2. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
The roadmap mentioned commitlint on commits but missed PR title validation. With squash-merge (now an explicit decision) the PR title becomes the commit on master and feeds release-please's changelog, so it must be linted too. Also explicit: local commit-msg hook and squash-only merge button. https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR establishes the Claude Code framework adoption plan for the repository and fixes several workflow configuration issues related to file path references.
Key Changes
Documentation
Added
docs/claude-framework-roadmap.md: Comprehensive roadmap documenting the Claude framework adoption across 6 phases (Phase 0–6), including:Updated
docs/dependencies-upgrades.md:bullseyetobookwormsecurity/directory)Workflow Fixes
.github/workflows/push-latest.yml:supported_platforms.jsontosecurity/**hashicorp.ascreference tosecurity/**pattern.github/workflows/build-test.yml:hashicorp.asctosecurity/**.github/workflows/dockerhub-description-update.yml:dockerhub-description.ymltodockerhub-description-update.yml(correct filename)Implementation Details
The roadmap establishes Phase 0 as a prerequisite cleanup phase, with these workflow fixes being part of that scope. The framework defines strict conventions for Claude sessions (no unauthorized merges, force-pushes, or direct master pushes) and establishes a phased delivery model with single PR per phase for focused review.
https://claude.ai/code/session_01PodDVUkYK2eRK86af5nUpA