Commit c4da67b
authored
refactor: rename SPIFFE_DEMO prefix to DOCSCLAW and fix workspace path (#95)
* refactor: rename SPIFFE_DEMO prefix to DOCSCLAW and fix workspace path
Replace all SPIFFE_DEMO-era naming artifacts with the current project
identity, and fix the default workspace path to be container-safe.
Breaking changes:
- Env var prefix SPIFFE_DEMO_* → DOCSCLAW_* (e.g. DOCSCLAW_OTEL_ENABLED)
- Config path /etc/spiffe-demo/ → /etc/docsclaw/
- Prometheus metric namespace spiffe_demo → docsclaw (all 7 metrics)
- Logger env var SPIFFE_DEMO_LOG_FORMAT → DOCSCLAW_LOG_FORMAT
Workspace fixes:
- Default workspace /tmp/agent-workspace → /workspace (safe with
readOnlyRootFilesystem: true in container hardening)
- System prompt now includes workspace path so the LLM writes files
to the correct location instead of guessing /tmp
- Deployment manifests updated: volume mount at /workspace,
agent-config.yaml workspace field updated to /workspace
Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>
Signed-off-by: Pavel Anni <panni@redhat.com>
* fix: complete workspace path rename to /workspace across all assets
Address review feedback on PR #95: the initial commit renamed the
default workspace in serve.go and the two primary deploy manifests,
but left /tmp/agent-workspace in all other user-facing assets.
Changes:
- containers/Containerfile.security: create /workspace (same 65532:0
owner, 775 mode); point TRIVY_CACHE_DIR and GRYPE_DB_CACHE_DIR
to /workspace/.cache/* so scanner databases land on the writable
emptyDir volume rather than the read-only image layer
- demo/batch/{finance,hr,security}: agent-config.yaml, deployment.yaml,
deployment-template.yaml, and *-config.yaml batch run configs
- demo/batch/dashboard/k8s.go: volume mount path
- examples/{nps-explorer,skill-playground}: configmap + deployment
- docs/demo/{hr-analyst,skill-discovery-test,executive-assistant}.yaml
- README.md, docs/agent-config-guide.md (field reference default),
site/getting-started.html
- testdata/research-agent/agent-config.yaml
- internal/workspace/workspace.go: update traversal-guard comment
example from /tmp/agent to /workspace/agent
Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>
Signed-off-by: Pavel Anni <panni@redhat.com>
* refactor: extract defaultWorkspace constant in serve.go
Replace the /workspace literal with a package-level constant so the
fallback value and the system prompt injection are defined in one place
and cannot drift independently.
Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>
Signed-off-by: Pavel Anni <panni@redhat.com>
---------
Signed-off-by: Pavel Anni <panni@redhat.com>1 parent 6b6b80e commit c4da67b
31 files changed
Lines changed: 295 additions & 54 deletions
File tree
- containers
- demo/batch
- dashboard
- finance
- hr
- security
- deploy
- docs
- demo
- examples
- nps-explorer
- skill-playground
- internal
- cmd
- config
- logger
- metrics
- workspace
- openspec/changes/rename-spiffe-and-fix-workspace
- site
- testdata/research-agent
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
170 | | - | |
| 170 | + | |
171 | 171 | | |
172 | 172 | | |
173 | 173 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | | - | |
122 | | - | |
123 | | - | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
124 | 124 | | |
125 | 125 | | |
126 | | - | |
127 | | - | |
| 126 | + | |
| 127 | + | |
128 | 128 | | |
129 | 129 | | |
130 | 130 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
568 | 568 | | |
569 | 569 | | |
570 | 570 | | |
571 | | - | |
| 571 | + | |
572 | 572 | | |
573 | 573 | | |
574 | 574 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | | - | |
| 64 | + | |
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
62 | | - | |
| 62 | + | |
63 | 63 | | |
64 | 64 | | |
65 | 65 | | |
| |||
136 | 136 | | |
137 | 137 | | |
138 | 138 | | |
139 | | - | |
| 139 | + | |
140 | 140 | | |
141 | 141 | | |
142 | 142 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
124 | 124 | | |
125 | 125 | | |
126 | 126 | | |
127 | | - | |
| 127 | + | |
128 | 128 | | |
129 | 129 | | |
130 | 130 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
0 commit comments