Conversation
Delete Released PVs after namespace deletion so new PVCs can bind to fresh PVs recreated by kustomize overlay. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…le timeouts Bundles four pipeline fixes surfaced by the 2026-05-18/19 openssl & wolfssl runs: - P1 (ast.py): _scan_include_dirs gains a 3-tier strategy — canonical include/src/source/lib first (OpenSSL), repo_root second (cJSON flat layout), then any first-level subdir containing headers (wolfssl/wolfssl/). _collect_source_files now skips build/CMakeFiles/_deps/etc. via a shared _is_build_artifact filter. - P2 (workflow_graph.py): new fix_build hotfix _try_hotfix_batch_internal_symbol_discovery resolves multiple `undefined reference` errors in one pass — scans apps/lib, src/lib, lib, core for matching function definitions and appends them to the SOURCES list of fuzz/build.py via _splice_sources_list. Toggled by SHERPA_FIX_BUILD_BATCH_DISCOVERY (default 1). - P3 (workflow_graph.py:5541): analysis idle timeout default 75s→300s, clamp ceiling 600s→1200s. Stops agents from being killed during long MCP / thinking pauses. - P4 (workflow_graph.py:9222, 15388, 15650): build subprocess timeouts 600s→1800s and synth validate timeout 90s→1800s. OpenSSL compiles take 5-8 min and were being SIGKILL'd at 600s. Also updates fix_build/SKILL.md to nudge the LLM fallback toward batch source addition, and refreshes CLAUDE.md env-var table + session state. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…raction) all_meta.update(meta_payload) was a shallow dict update: each file's 'functions' key completely overwrote the previous file's 'functions'. For cJSON, cjson_read_fuzzer.c (2 functions) processed last, overwriting cJSON.c's 120 functions — leaving only 2 in the final meta.json, all of which were fuzzer entry points, not cJSON API functions. Fix: deep-merge each top-level section so every file's functions/ composites/enums/typedefs are accumulated rather than replaced. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
PR Summary
Changed Files (top 200)
|
There was a problem hiding this comment.
Pull request overview
This PR tightens AST preprocessing heuristics for include-directory discovery and source-file collection, and improves build-repair behavior by increasing relevant timeouts and adding a batch “undefined reference” hotfix that can append multiple missing internal-library sources in one edit. It also updates operational documentation and the prod deploy reset workflow to clean up Released PVs.
Changes:
- Enhance AST preprocessing include-path scanning and exclude build-artifact directories/files.
- Extend workflow timeouts and add a batch internal-symbol discovery hotfix that edits
fuzz/build.pySOURCES in one pass. - Update fix_build skill guidance, session/env-var docs, and add Released-PV cleanup to prod reset.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| promefuzz-mcp/promefuzz_mcp/preprocessor/ast.py | Adds build-artifact filtering and multi-tier include-dir discovery/merging logic for AST preprocessing. |
| harness_generator/src/langchain_agent/workflow_graph.py | Adds SOURCES list splicing helper, batch undefined-symbol discovery hotfix, and increases several timeouts. |
| harness_generator/src/langchain_agent/opencode_skills/fix_build/SKILL.md | Updates fix_build workflow guidance to prefer batching undefined-reference fixes. |
| CLAUDE.md | Updates session state and documents new/changed env var defaults. |
| .github/workflows/deploy-prod.yml | Adds cleanup for Released PVs during prod reset before recreating the namespace. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| _push(repo_root / name) | ||
| _push(repo_root) | ||
| try: | ||
| for entry in sorted(repo_root.iterdir()): |
Comment on lines
+11834
to
+11839
| try: | ||
| c_count = sum(1 for _ in d.rglob("*.c")) | ||
| except (OSError, PermissionError): | ||
| continue | ||
| if c_count >= 3: | ||
| cand_dirs.append(d) |
| if [[ "${pv_name}" != *"${NAMESPACE}"* ]]; then | ||
| continue | ||
| fi | ||
| pv_status="$(kubectl get pv "${pv_name}" -o jsonpath='{.status.phase}')" |
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
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.
No description provided.