mirror of
https://github.com/affaan-m/everything-claude-code.git
synced 2026-04-05 08:43:29 +08:00
fix: correct TIMESTAMP environment variable syntax in observe.sh
The inline environment variable syntax `TIMESTAMP="$timestamp" echo ...` does not work correctly because: 1. The pipe creates a subshell that doesn't inherit the variable 2. The environment variable is set for echo, not for the piped python Fixed by using `export` and separating the commands: - export TIMESTAMP="$timestamp" - echo "$INPUT_JSON" | python3 -c "..." This ensures the TIMESTAMP variable is available to the python subprocess. Fixes #227 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -103,7 +103,8 @@ PARSED_OK=$(echo "$PARSED" | python3 -c "import json,sys; print(json.load(sys.st
|
|||||||
if [ "$PARSED_OK" != "True" ]; then
|
if [ "$PARSED_OK" != "True" ]; then
|
||||||
# Fallback: log raw input for debugging
|
# Fallback: log raw input for debugging
|
||||||
timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
|
timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
|
||||||
TIMESTAMP="$timestamp" echo "$INPUT_JSON" | python3 -c "
|
export TIMESTAMP="$timestamp"
|
||||||
|
echo "$INPUT_JSON" | python3 -c "
|
||||||
import json, sys, os
|
import json, sys, os
|
||||||
raw = sys.stdin.read()[:2000]
|
raw = sys.stdin.read()[:2000]
|
||||||
print(json.dumps({'timestamp': os.environ['TIMESTAMP'], 'event': 'parse_error', 'raw': raw}))
|
print(json.dumps({'timestamp': os.environ['TIMESTAMP'], 'event': 'parse_error', 'raw': raw}))
|
||||||
@@ -124,7 +125,8 @@ fi
|
|||||||
# Build and write observation
|
# Build and write observation
|
||||||
timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
|
timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
|
||||||
|
|
||||||
TIMESTAMP="$timestamp" echo "$PARSED" | python3 -c "
|
export TIMESTAMP="$timestamp"
|
||||||
|
echo "$PARSED" | python3 -c "
|
||||||
import json, sys, os
|
import json, sys, os
|
||||||
|
|
||||||
parsed = json.load(sys.stdin)
|
parsed = json.load(sys.stdin)
|
||||||
|
|||||||
Reference in New Issue
Block a user