@ainova-systems/intelligence 0.19.0 → 0.21.0

package/engine/adapters/opencode.sh

@@ -27,7 +27,9 @@
 # Every per-file loop batches its work into one awk process (see the batched
 # helpers in lib/common.sh): process spawns dominate sync time on Windows.
 
-source "$(dirname "${BASH_SOURCE[0]}")/../lib/common.sh"
+# The engine and the CLI load the shared library before any adapter. Only a
+# standalone source of this file loads it here: that costs a dirname process.
+declare -F finalize_output_file >/dev/null 2>&1 || source "$(dirname "${BASH_SOURCE[0]}")/../lib/common.sh"
 
 adapter_contract_opencode() {
     local output="${1%/}"
@@ -86,28 +88,24 @@ sync_opencode_commands() {
         [ "${#generated[@]}" -gt 0 ] && rm -f "${generated[@]}"
     fi
 
-    local src d skill_name
-    local -a skill_mds=()
-    load_yaml_list "$config_file" "skills"
-    local list="$IS_YAML_LIST"
-    while IFS= read -r src; do
-        [ -z "$src" ] && continue
-        local dir="$repo_root/$src"
-        [ -d "$dir" ] || continue
-        for d in "$dir"/*/; do
-            [ -d "$d" ] || continue
-            [ -f "$d/SKILL.md" ] || continue
-            skill_mds+=("${d%/}/SKILL.md")
-        done
-    done <<< "$list"
+    local skill_name
+    local -a skill_mds=() names=()
+    # The catalog's skills are the ones the inventory renders, symlinked ones
+    # included (decision 0017). The commands directory is tracked, so a local
+    # skill gets no command.
+    output_rel_var "$repo_root" "$commands_dir"
+    source_artifacts_var "$repo_root" "$config_file" skills opencode "$IS_OUTPUT_REL/{}.md"
+    if [ "${#IS_SOURCE_FILES[@]}" -gt 0 ]; then
+        skill_mds=("${IS_SOURCE_FILES[@]}")
+        names=("${IS_SOURCE_NAMES[@]}")
+    fi
 
     local count=0
     if [ "${#skill_mds[@]}" -gt 0 ]; then
         local path description argument_hint out_file
         local -a out_files=()
         while IFS=$'\x1f' read -r path description argument_hint; do
-            [ -n "$path" ] || continue
-            skill_name="${path%/SKILL.md}"; skill_name="${skill_name##*/}"
+            skill_name="${names[count]}"
             out_file="$commands_dir/$skill_name.md"
             yaml_dq_escape_var "$description"
             {
@@ -145,19 +143,13 @@ sync_opencode_agents() {
     local agents_dir="$output_dir/agents"
     mkdir -p "$agents_dir"
 
-    local src f
-    local -a files=()
-    load_yaml_list "$config_file" "agents"
-    local list="$IS_YAML_LIST"
-    while IFS= read -r src; do
-        [ -z "$src" ] && continue
-        local dir="$repo_root/$src"
-        [ -d "$dir" ] || continue
-        for f in "$dir"/*.md; do
-            [ -f "$f" ] || continue
-            files+=("$f")
-        done
-    done <<< "$list"
+    local -a files=() names=()
+    output_rel_var "$repo_root" "$agents_dir"
+    source_artifacts_var "$repo_root" "$config_file" agents opencode "$IS_OUTPUT_REL/{}.md"
+    if [ "${#IS_SOURCE_FILES[@]}" -gt 0 ]; then
+        files=("${IS_SOURCE_FILES[@]}")
+        names=("${IS_SOURCE_NAMES[@]}")
+    fi
 
     local count=0
     if [ "${#files[@]}" -gt 0 ]; then
@@ -165,8 +157,7 @@ sync_opencode_agents() {
 
         local path tier access description name edit_perm bash_perm spec="" report="" LS=$'\x1e'
         while IFS=$'\x1f' read -r path tier access description; do
-            [ -n "$path" ] || continue
-            name="${path##*/}"; name="${name%.md}"
+            name="${names[count]}"
             resolve_model_var "$tier"
 
             # Map source `access:` to opencode permission keys.

package/engine/adapters/pi.sh

@@ -4,6 +4,9 @@
 #
 # Rules:
 #   - Always-on (no `paths:`) -> SKIPPED here. Pi reads AGENTS.md natively.
+#     An always-on rule AGENTS.md does not carry — a local one, or one whose
+#     targets leave `agents` out — goes into the generated extension, which
+#     adds its body to the system prompt (decision 0021).
 #   - Path-scoped (with `paths:`) -> copied to .pi/intelligence-sync/rules/
 #     and surfaced via a generated Pi extension that lists them in the system
 #     prompt so the model can `read` the relevant rule on demand.
@@ -16,7 +19,9 @@
 # Every per-file loop batches its work into one awk process (see the batched
 # helpers in lib/common.sh): process spawns dominate sync time on Windows.
 
-source "$(dirname "${BASH_SOURCE[0]}")/../lib/common.sh"
+# The engine and the CLI load the shared library before any adapter. Only a
+# standalone source of this file loads it here: that costs a dirname process.
+declare -F finalize_output_file >/dev/null 2>&1 || source "$(dirname "${BASH_SOURCE[0]}")/../lib/common.sh"
 
 adapter_contract_pi() {
     local output="${1%/}"
@@ -83,6 +88,39 @@ pi_rule_paths_rows() {
     ' "$@"
 }
 
+# pi_rule_body_rows <file>... — one line per file in argument order: its body,
+# without the frontmatter and leading or trailing blank lines, escaped as the
+# contents of a TypeScript double-quoted string (`\`, `"`, tabs and line breaks
+# escaped).
+pi_rule_body_rows() {
+    [ "$#" -gt 0 ] || return 0
+    awk "$IS_AWK_LIB"'
+        function store(   j, first, last, s) {
+            if (cur == "") return
+            first = 0; last = 0
+            for (j = 1; j <= n; j++) if (B[j] != "") { if (!first) first = j; last = j }
+            s = ""
+            for (j = first; first && j <= last; j++) s = s (j > first ? "\\n" : "") B[j]
+            R[cur] = s
+        }
+        FNR == 1 { store(); cur = FILENAME; n = 0; in_fm = 0; past_fm = 0 }
+        { sub(/\r$/, "") }
+        FNR == 1 && $0 != "---" { past_fm = 1 }
+        /^---$/ && !past_fm { in_fm = !in_fm; if (!in_fm) past_fm = 1; next }
+        !past_fm { next }
+        {
+            line = is_repl($0, "\\", "\\\\")
+            line = is_repl(line, "\"", "\\\"")
+            line = is_repl(line, "\t", "\\t")
+            B[++n] = line
+        }
+        END {
+            store()
+            for (i = 1; i < ARGC; i++) print (ARGV[i] in R) ? R[ARGV[i]] : ""
+        }
+    ' "$@"
+}
+
 sync_pi_rules() {
     local repo_root="$1"
     local config_file="$2"
@@ -96,38 +134,44 @@ sync_pi_rules() {
 
     mkdir -p "$rules_root" "$(dirname "$extension_file")"
 
-    local src f
-    local -a files=()
-    load_yaml_list "$config_file" "rules"
-    local list="$IS_YAML_LIST"
-    while IFS= read -r src; do
-        [ -z "$src" ] && continue
-        local dir="$repo_root/$src"
-        [ -d "$dir" ] || continue
-        for f in "$dir"/*.md; do
-            [ -f "$f" ] || continue
-            files+=("$f")
-        done
-    done <<< "$list"
-
-    local -a scoped=()
-    if [ "${#files[@]}" -gt 0 ]; then
-        local path hp
-        while IFS=$'\x1f' read -r path hp; do
-            [ -n "$path" ] || continue
-            [ "$hp" -eq 0 ] && continue
-            scoped+=("$path")
-        done < <(frontmatter_index "paths#" "${files[@]}")
+    local -a scoped=() scoped_names=() always=() always_names=()
+    local k=0 always_rules=""
+    output_rel_var "$repo_root" "$rules_root"
+    source_artifacts_var "$repo_root" "$config_file" rules pi "$IS_OUTPUT_REL/{}.md"
+    while [ "$k" -lt "${#IS_SOURCE_FILES[@]}" ]; do
+        if [ "${IS_SOURCE_SCOPED[k]}" = 1 ]; then
+            scoped+=("${IS_SOURCE_FILES[k]}")
+            scoped_names+=("${IS_SOURCE_NAMES[k]}")
+        elif [ "${IS_SOURCE_IN_AGENTS[k]}" = 0 ]; then
+            # An always-on rule AGENTS.md does not carry: the extension adds
+            # its body to the system prompt (decision 0021).
+            always+=("${IS_SOURCE_FILES[k]}")
+            always_names+=("${IS_SOURCE_NAMES[k]}")
+        fi
+        k=$((k + 1))
+    done
+
+    if [ "${#always[@]}" -gt 0 ]; then
+        local body
+        k=0
+        while IFS= read -r body; do
+            pi_ts_escape_var "${always_names[k]}"
+            always_rules+="    { name: \"$IS_PI_TS_ESCAPED\", body: \"$body\" },"$'\n'
+            echo "  rule: ${always_names[k]} (always-on, in the extension)"
+            k=$((k + 1))
+        done < <(pi_rule_body_rows "${always[@]}")
     fi
 
     if [ "${#scoped[@]}" -gt 0 ]; then
-        finalize_copy_files "$rules_root" "${scoped[@]}"
+        finalize_copy_named "$rules_root" "$IS_SOURCE_NAME_MAP" "${scoped[@]}"
 
         local -a row_arr
         local base rel_rule_path patterns_js display sep display_sep pat i
+        k=0
         while IFS=$'\x1f' read -r -a row_arr; do
             [ "${#row_arr[@]}" -gt 0 ] || continue
-            base="${row_arr[0]##*/}"
+            base="${scoped_names[k]}.md"
+            k=$((k + 1))
             rel_rule_path="$output_rel/intelligence-sync/rules/$base"
             patterns_js=""
             display=""
@@ -151,7 +195,64 @@ sync_pi_rules() {
         done < <(pi_rule_paths_rows "${scoped[@]}")
     fi
 
-    if [ "$count" -gt 0 ]; then
+    if [ "${#always[@]}" -gt 0 ]; then
+        {
+            cat <<'EOF'
+/**
+ * Generated by intelligence-sync. Do not edit manually.
+ * Adds the always-on project rules AGENTS.md does not carry to the system
+ * prompt, and lists path-scoped project rules generated from intelligence/rules/.
+ */
+
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+
+const projectRules: Array<{ name: string; body: string }> = [
+EOF
+            printf '%s' "$always_rules"
+            cat <<'EOF'
+];
+
+const scopedRules: Array<{ path: string; patterns: string[] }> = [
+EOF
+            printf '%s' "$manifest"
+            cat <<'EOF'
+];
+
+export default function intelligenceSyncRulesExtension(pi: ExtensionAPI) {
+    pi.on("before_agent_start", async (event) => {
+        let systemPrompt = event.systemPrompt;
+
+        if (projectRules.length > 0) {
+            systemPrompt += `
+
+## Project Rules
+
+${projectRules.map((rule) => rule.body).join("\n\n")}`;
+        }
+
+        if (scopedRules.length > 0) {
+            const rulesList = scopedRules
+                .map((rule) => `- ${rule.path} — paths: ${rule.patterns.join(", ")}`)
+                .join("\n");
+
+            systemPrompt += `
+
+## Scoped Project Rules
+
+AGENTS.md already carries the always-on project rules. Additional path-scoped rules are available on demand:
+
+${rulesList}
+
+When the task touches files that match one of these path patterns, use the read tool to load the relevant rule file before making changes.`;
+        }
+
+        return { systemPrompt };
+    });
+}
+EOF
+        } > "$extension_file"
+        finalize_output_file "$extension_file"
+    elif [ "$count" -gt 0 ]; then
         {
             cat <<'EOF'
 /**
@@ -197,7 +298,11 @@ EOF
         finalize_output_file "$extension_file"
     fi
 
-    echo "  -> Rules: $count scoped"
+    if [ "${#always[@]}" -gt 0 ]; then
+        echo "  -> Rules: $count scoped, ${#always[@]} always-on"
+    else
+        echo "  -> Rules: $count scoped"
+    fi
 }
 
 sync_pi_skills() {
@@ -221,25 +326,18 @@ sync_pi_agents() {
 
     mkdir -p "$prompts_dir"
 
-    local src f
-    local -a files=()
-    load_yaml_list "$config_file" "agents"
-    local list="$IS_YAML_LIST"
-    while IFS= read -r src; do
-        [ -z "$src" ] && continue
-        local dir="$repo_root/$src"
-        [ -d "$dir" ] || continue
-        for f in "$dir"/*.md; do
-            [ -f "$f" ] || continue
-            files+=("$f")
-        done
-    done <<< "$list"
+    local -a files=() names=()
+    output_rel_var "$repo_root" "$prompts_dir"
+    source_artifacts_var "$repo_root" "$config_file" agents pi "$IS_OUTPUT_REL/intelligence-agent-{}.md"
+    if [ "${#IS_SOURCE_FILES[@]}" -gt 0 ]; then
+        files=("${IS_SOURCE_FILES[@]}")
+        names=("${IS_SOURCE_NAMES[@]}")
+    fi
 
     if [ "${#files[@]}" -gt 0 ]; then
         local path desc access name template_desc spec="" report="" LS=$'\x1e'
         while IFS=$'\x1f' read -r path desc access; do
-            [ -n "$path" ] || continue
-            name="${path##*/}"; name="${name%.md}"
+            name="${names[count]}"
             repo_rel_link_var "$repo_root" "$path"
 
             template_desc="Use $name persona"

package/engine/lib/adapter-contract.sh

@@ -122,7 +122,7 @@ adapter_contract_safe_concrete_path() {
 # into the caller. Project adapters are trusted executable code during sync;
 # the isolation here is for correctness, not a security boundary.
 adapter_contract_records() (
-    local name="$1" file="$2" output="$3" manifest="${4:-}" fn line kind value saw_version=0
+    local name="$1" file="$2" output="$3" manifest="${4:-}" fn
     # shellcheck source=/dev/null
     source "$file"
     printf -v fn 'adapter_contract_%s' "$name"
@@ -130,6 +130,38 @@ adapter_contract_records() (
         echo "ERROR: adapter '$name' has no $fn contract" >&2
         return 1
     }
+    _adapter_contract_emit "$name" '' < <("$fn" "$output" "$manifest")
+)
+
+# adapter_contract_records_batch <scratch-file> <name> <file> <output>... —
+# adapter_contract_records for several built-in adapters in one subshell, each
+# record prefixed with "<name><TAB>". One process instead of two per adapter:
+# on Git Bash every fork costs tens of milliseconds. <scratch-file> receives
+# each contract's raw lines before they are checked.
+adapter_contract_records_batch() (
+    local scratch="$1" name file output fn
+    shift
+    while [ "$#" -ge 3 ]; do
+        name="$1" file="$2" output="$3"
+        shift 3
+        # shellcheck source=/dev/null
+        source "$file"
+        printf -v fn 'adapter_contract_%s' "$name"
+        declare -F "$fn" >/dev/null 2>&1 || {
+            echo "ERROR: adapter '$name' has no $fn contract" >&2
+            return 1
+        }
+        # Like the process substitution above, a contract's own status is not
+        # its verdict: the records it printed are.
+        "$fn" "$output" > "$scratch" || true
+        _adapter_contract_emit "$name" "$name"$'\t' < "$scratch" || return 1
+    done
+)
+
+# _adapter_contract_emit <adapter-name> <prefix> — check the contract records
+# on stdin and print each one after <prefix>; return 1 on the first bad one.
+_adapter_contract_emit() {
+    local name="$1" prefix="$2" line kind value saw_version=0
     while IFS= read -r line; do
         [ -n "$line" ] || continue
         kind="${line%%$'\t'*}"
@@ -171,10 +203,10 @@ adapter_contract_records() (
                 return 1
                 ;;
         esac
-        printf '%s\n' "$line"
-    done < <("$fn" "$output" "$manifest")
+        printf '%s%s\n' "$prefix" "$line"
+    done
     [ "$saw_version" -eq 1 ] || {
         echo "ERROR: adapter '$name' contract did not declare version 1" >&2
         return 1
     }
-)
+}