@guanlin4924/devflow-pi 0.1.93 → 0.1.95

package/plugins/devflow-core/skills/workflow-orchestrator/scripts/validate_state.rb

@@ -44,10 +44,23 @@ module WorkflowStateValidator
     workflow_step_unknown
     phase_ownership_conflict
   ].freeze
+  # Acceptance-execution trust failures are never downgraded by soft gates.
+  ACCEPTANCE_HARD_VALIDATION_CODES = %w[
+    acceptance_results_not_passed
+    execution_surface_mismatch
+    approval_invalid
+    human_result_unapproved
+    dsl_not_approved
+    environment_profile_not_approved
+    runtime_integrity_error
+    acceptance_tests_not_passed
+    acceptance_tests_empty
+  ].freeze
 
   def self.hard_validation_code?(code)
     value = code.to_s
     return true if HARD_VALIDATION_CODES.include?(value)
+    return true if ACCEPTANCE_HARD_VALIDATION_CODES.include?(value)
     return true if value.start_with?("artifact_registry_", "parallel_", "PARALLEL_")
     return true if value.start_with?("duplicate_", "unknown_", "missing_tracked_step")
     return true if value.start_with?("step_in_", "completed_", "terminal_", "non_completed_")
@@ -61,9 +74,9 @@ module WorkflowStateValidator
 
   def validate(root:, state_path:, state_override: nil, workflow_override: nil, artifact_registry_override: nil, strict_artifact_status: true, strict_registry_revision: true, phase_owner_override: nil)
     root = File.expand_path(root)
-    canonical_state_path, = DevflowControlIO.resolve_control_path(root, ".planning/STATE.md", :state)
+    canonical_state_path, = DevflowControlIO.resolve_control_path(root, ".devflow/planning/STATE.md", :state)
     unless File.expand_path(state_path, root) == canonical_state_path
-      issues = [issue("error", "invalid_control_path", "STATE.md path must be .planning/STATE.md")]
+      issues = [issue("error", "invalid_control_path", "STATE.md path must be .devflow/planning/STATE.md")]
       return result(false, state_path, nil, nil, issues)
     end
     state_path = canonical_state_path
@@ -84,7 +97,7 @@ module WorkflowStateValidator
       return result(false, state_path, nil, nil, issues)
     end
 
-    workflow_path = state["workflow_config"] || File.join(".planning", "workflows", "#{state.fetch("current_flow")}.yaml")
+    workflow_path = state["workflow_config"] || File.join(".devflow/planning", "workflows", "#{state.fetch("current_flow")}.yaml")
     abs_workflow_path, workflow_path = DevflowControlIO.resolve_control_path(root, workflow_path, :workflow)
     workflow = workflow_override || load_yaml(abs_workflow_path, issues, "workflow_config")
     return result(false, state_path, workflow_path, nil, issues) unless workflow
@@ -806,7 +819,7 @@ unless %w[prepared claimed consumed superseded failed].include?(status)
     return unless %w[frontend-tech-spec backend-tech-spec].include?(step["id"])
 
     feature = state["current_feature"]
-    brief_path = File.join(root, ".planning", "features", feature, "architecture-brief.meta.json")
+    brief_path = File.join(root, ".devflow/planning", "features", feature, "architecture-brief.meta.json")
     unless File.file?(brief_path)
       issues << issue("error", "architecture_brief_meta_missing", "Technical spec cannot be validated without the architecture brief meta", step: step["id"], path: brief_path)
       return
@@ -826,7 +839,7 @@ unless %w[prepared claimed consumed superseded failed].include?(status)
 
     graph, = RequirementGraph.load_graph(root, feature)
     if brief["graph_revision"] != graph["revision"]
-      brief_markdown_path = File.join(root, ".planning", "features", feature, "architecture-brief.md")
+      brief_markdown_path = File.join(root, ".devflow/planning", "features", feature, "architecture-brief.md")
       current_markdown = ArchitectureBrief.build(root, feature)[3]
       normalize_revision = lambda do |markdown|
         markdown.to_s.gsub(/需求图谱：schema v(\d+) \/ revision \d+/, '需求图谱：schema v\1 / revision <current>')
@@ -1022,7 +1035,7 @@ unless %w[prepared claimed consumed superseded failed].include?(status)
 end
 
 if __FILE__ == $PROGRAM_NAME
-  options = { state: ".planning/STATE.md", strict_artifact_status: true, json: false }
+  options = { state: ".devflow/planning/STATE.md", strict_artifact_status: true, json: false }
   OptionParser.new do |opts|
     opts.banner = "Usage: validate_state.rb [--state PATH] [--no-strict-artifact-status] [--json]"
     opts.on("--state PATH", "STATE.md path") { |path| options[:state] = path }

package/plugins/devflow-core/skills/workflow-orchestrator/scripts/wireframe_commit.rb

@@ -16,7 +16,7 @@ options = {
   action: "meta",
   feature: nil,
   canonical: nil,
-  state: ".planning/STATE.md",
+  state: ".devflow/planning/STATE.md",
   expected_revision: nil,
   confirmed_screens: nil,
   approved_by: nil,
@@ -107,7 +107,7 @@ def canonical_fingerprint(canonical)
 end
 
 def read_source_fingerprint(root, feature)
-  path = File.join(root, ".planning", "requirements", feature, "traceability.json")
+  path = File.join(root, ".devflow/planning", "requirements", feature, "traceability.json")
   return "" unless File.file?(path)
 
   JSON.parse(File.read(path))["source_document_fingerprint"].to_s
@@ -151,7 +151,7 @@ begin
     raise ArgumentError, "--runtime-confirmed is required for approval" unless options[:runtime_confirmed]
   end
 
-  canonical_relative = options[:canonical] || ".planning/features/#{options[:feature]}/canonical.json"
+  canonical_relative = options[:canonical] || ".devflow/planning/features/#{options[:feature]}/canonical.json"
   state_absolute, = DevflowControlIO.resolve_control_path(ROOT, options[:state], :state)
   canonical_absolute, canonical_path = DevflowControlIO.resolve_control_path(ROOT, canonical_relative, :artifact)
   raise ArgumentError, "canonical.json does not exist" unless File.file?(canonical_absolute)
@@ -176,7 +176,7 @@ begin
   raise ArgumentError, "source_document_fingerprint is required" if source_fingerprint.empty?
 
   screen_ids = Array(canonical["screens"]).map { |screen| screen["id"].to_s }.sort
-  meta_path = File.join(ROOT, ".planning", "features", options[:feature], "canonical.meta.json")
+  meta_path = File.join(ROOT, ".devflow/planning", "features", options[:feature], "canonical.meta.json")
   meta = {
     "schema_version" => 1,
     "feature" => options[:feature],
@@ -208,7 +208,7 @@ begin
       "approved_by" => options[:approved_by],
       "approved_at" => Time.now.utc.iso8601
     }
-    approval_path = File.join(ROOT, ".planning", "features", options[:feature], "approval.json")
+    approval_path = File.join(ROOT, ".devflow/planning", "features", options[:feature], "approval.json")
     DevflowControlIO.atomic_write(approval_path, JSON.pretty_generate(approval) + "\n")
     output["action"] = "approve"
     output["approval"] = approval

package/plugins/devflow-core/skills/workflow-orchestrator/scripts/wireframe_validate.py

@@ -451,7 +451,7 @@ def main() -> int:
     errors, canonical = validate(proposal, traceability, schema, business_model)
     questions_path = args.questions
     if questions_path is None and isinstance(proposal, dict) and proposal.get("feature"):
-        questions_path = Path(".planning") / "features" / str(proposal["feature"]) / "questions.yaml"
+        questions_path = Path(".devflow/planning") / "features" / str(proposal["feature"]) / "questions.yaml"
     question_meta: dict[str, Any]
     blocking_open: list[str] = []
     if questions_path is None or not questions_path.exists():

package/plugins/devflow-core/skills/workflow-orchestrator/templates/acceptance-remediation.yaml

@@ -10,7 +10,7 @@ entry:
   initial_step: diagnose
 
 state_contract:
-  source: ".planning/STATE.md"
+  source: ".devflow/planning/STATE.md"
   suggested_transition:
     current_flow: acceptance-remediation
     current_step: diagnose
@@ -35,10 +35,10 @@ steps:
     skill: workflow-bug-fix
     depends_on: []
     inputs:
-      - ".planning/test-runs/{feature}/acceptance-results.json"
-      - ".planning/features/{feature}/acceptance-test-bindings.json"
+      - ".devflow/planning/test-runs/{feature}/acceptance-results.json"
+      - ".devflow/planning/features/{feature}/acceptance-test-bindings.json"
     outputs:
-      - path: ".planning/features/{feature}/acceptance-remediation.md"
+      - path: ".devflow/planning/features/{feature}/acceptance-remediation.md"
         type: acceptance-remediation
         required: true
 
@@ -49,8 +49,8 @@ steps:
     depends_on:
       - diagnose
     inputs:
-      - ".planning/features/{feature}/acceptance-remediation.md"
-      - ".planning/features/{feature}/变更清单.md"
+      - ".devflow/planning/features/{feature}/acceptance-remediation.md"
+      - ".devflow/planning/features/{feature}/变更清单.md"
 
   - id: code-index-refresh
     order: 30
@@ -59,9 +59,9 @@ steps:
     depends_on:
       - execute-fix
     inputs:
-      - ".planning/features/{feature}/acceptance-remediation.md"
+      - ".devflow/planning/features/{feature}/acceptance-remediation.md"
     outputs:
-      - path: ".planning/code-index/AUDIT.md"
+      - path: ".devflow/planning/code-index/AUDIT.md"
         type: code-index-audit
         required: true
 
@@ -72,11 +72,11 @@ steps:
     depends_on:
       - code-index-refresh
     inputs:
-      - ".planning/requirements/{feature}/tests.json"
-      - ".planning/features/{feature}/acceptance-test-bindings.json"
-      - ".planning/code-index/AUDIT.md"
+      - ".devflow/planning/requirements/{feature}/tests.json"
+      - ".devflow/planning/features/{feature}/acceptance-test-bindings.json"
+      - ".devflow/planning/code-index/AUDIT.md"
     outputs:
-      - path: ".planning/test-runs/{feature}/acceptance-results.json"
+      - path: ".devflow/planning/test-runs/{feature}/acceptance-results.json"
         type: acceptance-test-results
         required: true