Repository navigation
Add AGENTS.md with observability guidance - #24
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Petra Jaros <petra@fil.org> Claude-Session: https://claude.ai/code/session_01JCF8gyb8jNRmHmWqQmVhRJ
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Petra Jaros <petra@fil.org> Claude-Session: https://claude.ai/code/session_01JCF8gyb8jNRmHmWqQmVhRJ
|
Claude reviewer says… (1st round, Approve once one should-fix is done. Verified across the six
Findings
Generated by Claude Code |
|
Claude author says… Fixed in
Generated by Claude Code |
|
Claude reviewer says… (2nd round, Approve. Both round 1 should-fix findings are fixed, and only nits remain. Verified
Findings
Generated by Claude Code |
|
Claude author says…
This round found only nits, so there won't be another. Generated by Claude Code |
Following bajtos's review of fil-forge/infra-nodes#174: the guidance file loads into every session, so its Observability section is now a short reminder of what to add and when to skip it, with a pointer to docs/observability.md, which holds the detail. A CLAUDE.md that only imported AGENTS.md is gone where this PR added one: Claude Code reads AGENTS.md itself from v2.1.277. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Petra Jaros <petra@fil.org> Claude-Session: https://claude.ai/code/session_01JCF8gyb8jNRmHmWqQmVhRJ
Review round on the guidance restructure: some paragraphs were never reflowed after the pointer edit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Petra Jaros <petra@fil.org> Claude-Session: https://claude.ai/code/session_01JCF8gyb8jNRmHmWqQmVhRJ
|
Claude reviewer says… (round on the restructure, This round reviewed all eight guidance PRs together; findings below are this PR's. Approve. Only nits here, and none of the eight PRs has anything blocking or should-fix. Verified, across all eight PRs
Findings for this PR
Generated by Claude Code |
|
Claude author says… The fixes are in
Check: This round found only nits, so there won't be another. Generated by Claude Code |
Published from fil-forge/swarf#24 - Digest: `sha256:4807de8a7dd8c81ea26154447affebf5d912d1ea00bbd2e911fcc222950f4098` - Commit: fil-forge/swarf@974851a - Publish run: https://github.com/fil-forge/swarf/actions/runs/38970560570 Merging applies [`terraform/envs/dev/apps`](https://github.com/fil-forge/infra-central/tree/main/terraform/envs/dev/apps) with no further confirmation.
Published from fil-forge/swarf#24 - Digest: `sha256:4807de8a7dd8c81ea26154447affebf5d912d1ea00bbd2e911fcc222950f4098` - Commit: fil-forge/swarf@974851a - Publish run: https://github.com/fil-forge/swarf/actions/runs/38970560570 Merging applies [`terraform/envs/dev/apps`](https://github.com/fil-forge/infra-central/tree/main/terraform/envs/dev/apps) with no further confirmation.
Published from fil-forge/swarf#24 - Digest: `sha256:4807de8a7dd8c81ea26154447affebf5d912d1ea00bbd2e911fcc222950f4098` - Commit: fil-forge/swarf@974851a - Publish run: https://github.com/fil-forge/swarf/actions/runs/38970560570 Merging applies [`terraform/envs/dev/apps`](https://github.com/fil-forge/infra-central/tree/main/terraform/envs/dev/apps) with no further confirmation. Co-authored-by: fil-forge-bot[bot] <318653112+fil-forge-bot[bot]@users.noreply.github.com>
Sets every staging pin in `terraform/envs/staging/apps/terraform.tfvars` to what `terraform/envs/dev/apps/terraform.tfvars` pins at f312ede. Merging this is the promotion: Check and deploy applies `terraform/envs/staging/apps` on the merge commit, with no further confirmation. dev passed its smoke test on that commit in https://github.com/fil-forge/infra-central/actions/runs/38971842486. This pull request is rebuilt each time dev passes its smoke test after a push to `main`, and closed once staging matches dev. To hold a service back, push to `bot/promote-staging` by hand: the bot leaves the branch alone while this pull request is open. ## sprue [`sha256:79eb33d`](https://github.com/orgs/fil-forge/packages/container/sprue/1339893466) → [`sha256:deab582`](https://github.com/orgs/fil-forge/packages/container/sprue/1399561986) What fil-forge/sprue brings over staging's pin, [`9b40689..88ec247`](fil-forge/sprue@9b40689...88ec247): - fil-forge/sprue#130 ([0b76576](fil-forge/sprue@0b76576)) - fil-forge/sprue#131 ([8c7925a](fil-forge/sprue@8c7925a)) - fil-forge/sprue#132 ([a7762f6](fil-forge/sprue@a7762f6)) - fil-forge/sprue#133 ([a5d84ee](fil-forge/sprue@a5d84ee)) - fil-forge/sprue#137 ([98d22cc](fil-forge/sprue@98d22cc)) - fil-forge/sprue#136 ([9fb2d96](fil-forge/sprue@9fb2d96)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([dbd600d](fil-forge/sprue@dbd600d)) - `Address round 1 review of the observability guidance` ([4188af6](fil-forge/sprue@4188af6)) - fil-forge/sprue#141 ([046155c](fil-forge/sprue@046155c)) - fil-forge/sprue#142 ([80c2ed6](fil-forge/sprue@80c2ed6)) - fil-forge/sprue#145 ([9e5422e](fil-forge/sprue@9e5422e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([a5ee6c7](fil-forge/sprue@a5ee6c7)) - fil-forge/sprue#143 ([3c62127](fil-forge/sprue@3c62127)) - `Rewrap docs/observability.md` ([5c8387a](fil-forge/sprue@5c8387a)) - fil-forge/sprue#140 ([88ec247](fil-forge/sprue@88ec247)) ## hilt [`sha256:5b6d022`](https://github.com/orgs/fil-forge/packages/container/hilt/1384674731) → [`sha256:9ba1568`](https://github.com/orgs/fil-forge/packages/container/hilt/1386299607) What fil-forge/hilt brings over staging's pin, [`42dd070..b5d3b3e`](fil-forge/hilt@42dd070...b5d3b3e): - fil-forge/hilt#97 ([b5d3b3e](fil-forge/hilt@b5d3b3e)) ## swarf [`sha256:c364312`](https://github.com/orgs/fil-forge/packages/container/swarf/1361661163) → [`sha256:4807de8`](https://github.com/orgs/fil-forge/packages/container/swarf/1399568771) What fil-forge/swarf brings over staging's pin, [`4b2e2ae..974851a`](fil-forge/swarf@4b2e2ae...974851a): - `Add AGENTS.md with observability guidance` ([88dbb4c](fil-forge/swarf@88dbb4c)) - `Address round 1 review of the observability guidance` ([305021e](fil-forge/swarf@305021e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([7ff3da0](fil-forge/swarf@7ff3da0)) - `Rewrap docs/observability.md` ([72018b4](fil-forge/swarf@72018b4)) - fil-forge/swarf#24 ([974851a](fil-forge/swarf@974851a))
Sets every staging pin in `terraform/envs/staging/apps/terraform.tfvars` to what `terraform/envs/dev/apps/terraform.tfvars` pins at fcd18a1. Merging this is the promotion: Check and deploy applies `terraform/envs/staging/apps` on the merge commit, with no further confirmation. dev passed its smoke test on that commit in https://github.com/fil-forge/infra-central/actions/runs/38972429322. This pull request is rebuilt each time dev passes its smoke test after a push to `main`, and closed once staging matches dev. To hold a service back, push to `bot/promote-staging` by hand: the bot leaves the branch alone while this pull request is open. ## sprue [`sha256:79eb33d`](https://github.com/orgs/fil-forge/packages/container/sprue/1339893466) → [`sha256:deab582`](https://github.com/orgs/fil-forge/packages/container/sprue/1399561986) What fil-forge/sprue brings over staging's pin, [`9b40689..88ec247`](fil-forge/sprue@9b40689...88ec247): - fil-forge/sprue#130 ([0b76576](fil-forge/sprue@0b76576)) - fil-forge/sprue#131 ([8c7925a](fil-forge/sprue@8c7925a)) - fil-forge/sprue#132 ([a7762f6](fil-forge/sprue@a7762f6)) - fil-forge/sprue#133 ([a5d84ee](fil-forge/sprue@a5d84ee)) - fil-forge/sprue#137 ([98d22cc](fil-forge/sprue@98d22cc)) - fil-forge/sprue#136 ([9fb2d96](fil-forge/sprue@9fb2d96)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([dbd600d](fil-forge/sprue@dbd600d)) - `Address round 1 review of the observability guidance` ([4188af6](fil-forge/sprue@4188af6)) - fil-forge/sprue#141 ([046155c](fil-forge/sprue@046155c)) - fil-forge/sprue#142 ([80c2ed6](fil-forge/sprue@80c2ed6)) - fil-forge/sprue#145 ([9e5422e](fil-forge/sprue@9e5422e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([a5ee6c7](fil-forge/sprue@a5ee6c7)) - fil-forge/sprue#143 ([3c62127](fil-forge/sprue@3c62127)) - `Rewrap docs/observability.md` ([5c8387a](fil-forge/sprue@5c8387a)) - fil-forge/sprue#140 ([88ec247](fil-forge/sprue@88ec247)) ## hilt [`sha256:5b6d022`](https://github.com/orgs/fil-forge/packages/container/hilt/1384674731) → [`sha256:94be06b`](https://github.com/orgs/fil-forge/packages/container/hilt/1399559133) What fil-forge/hilt brings over staging's pin, [`42dd070..1f0b2c1`](fil-forge/hilt@42dd070...1f0b2c1): - fil-forge/hilt#97 ([b5d3b3e](fil-forge/hilt@b5d3b3e)) - fil-forge/hilt#98 ([1f0b2c1](fil-forge/hilt@1f0b2c1)) ## swarf [`sha256:c364312`](https://github.com/orgs/fil-forge/packages/container/swarf/1361661163) → [`sha256:4807de8`](https://github.com/orgs/fil-forge/packages/container/swarf/1399568771) What fil-forge/swarf brings over staging's pin, [`4b2e2ae..974851a`](fil-forge/swarf@4b2e2ae...974851a): - `Add AGENTS.md with observability guidance` ([88dbb4c](fil-forge/swarf@88dbb4c)) - `Address round 1 review of the observability guidance` ([305021e](fil-forge/swarf@305021e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([7ff3da0](fil-forge/swarf@7ff3da0)) - `Rewrap docs/observability.md` ([72018b4](fil-forge/swarf@72018b4)) - fil-forge/swarf#24 ([974851a](fil-forge/swarf@974851a)) ## delegator [`sha256:30b1757`](https://github.com/orgs/fil-forge/packages/container/delegator/1178963957) → [`sha256:7fb4eb5`](https://github.com/orgs/fil-forge/packages/container/delegator/1399567101) What fil-forge/delegator brings over staging's pin, [`b5552f1..3ae7270`](fil-forge/delegator@b5552f1...3ae7270): - fil-forge/delegator#35 ([8bf4153](fil-forge/delegator@8bf4153)) - fil-forge/delegator#36 ([3dfe8c4](fil-forge/delegator@3dfe8c4)) - fil-forge/delegator#37 ([eb81da2](fil-forge/delegator@eb81da2)) - fil-forge/delegator#38 ([314b6d6](fil-forge/delegator@314b6d6)) - fil-forge/delegator#40 ([e588b27](fil-forge/delegator@e588b27)) - fil-forge/delegator#42 ([9cfc334](fil-forge/delegator@9cfc334)) - fil-forge/delegator#43 ([3d3525c](fil-forge/delegator@3d3525c)) - fil-forge/delegator#45 ([5e4090c](fil-forge/delegator@5e4090c)) - fil-forge/delegator#47 ([22d663d](fil-forge/delegator@22d663d)) - fil-forge/delegator#44 ([984ea6e](fil-forge/delegator@984ea6e)) - fil-forge/delegator#46 ([861f107](fil-forge/delegator@861f107)) - fil-forge/delegator#48 ([5a14d35](fil-forge/delegator@5a14d35)) - fil-forge/delegator#49 ([3fe2769](fil-forge/delegator@3fe2769)) - fil-forge/delegator#52 ([1ee1e5b](fil-forge/delegator@1ee1e5b)) - fil-forge/delegator#51 ([d12f29a](fil-forge/delegator@d12f29a)) - fil-forge/delegator#53 ([8d3b969](fil-forge/delegator@8d3b969)) - fil-forge/delegator#55 ([4d6e592](fil-forge/delegator@4d6e592)) - fil-forge/delegator#56 ([4d37521](fil-forge/delegator@4d37521)) - fil-forge/delegator#59 ([26d9d4e](fil-forge/delegator@26d9d4e)) - fil-forge/delegator#60 ([7eeff65](fil-forge/delegator@7eeff65)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([fe04a4c](fil-forge/delegator@fe04a4c)) - `Address round 1 review of the observability guidance` ([b29fad9](fil-forge/delegator@b29fad9)) - fil-forge/delegator#62 ([aaba55d](fil-forge/delegator@aaba55d)) - fil-forge/delegator#63 ([b3bc907](fil-forge/delegator@b3bc907)) - fil-forge/delegator#64 ([716049e](fil-forge/delegator@716049e)) - fil-forge/delegator#66 ([2c2784d](fil-forge/delegator@2c2784d)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([409ff15](fil-forge/delegator@409ff15)) - `Rewrap docs/observability.md` ([77e9d0a](fil-forge/delegator@77e9d0a)) - fil-forge/delegator#61 ([3ae7270](fil-forge/delegator@3ae7270)) ## signing_service [`sha256:b7ef5f0`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1157634158) → [`sha256:4493534`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1399574922) What fil-forge/piri-signing-service brings over staging's pin, [`c2ac7db..678c9c1`](fil-forge/piri-signing-service@c2ac7db...678c9c1): - fil-forge/piri-signing-service#19 ([8472ad9](fil-forge/piri-signing-service@8472ad9)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([53d3758](fil-forge/piri-signing-service@53d3758)) - `Address round 1 review of the observability guidance` ([950ef02](fil-forge/piri-signing-service@950ef02)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([e07d0ee](fil-forge/piri-signing-service@e07d0ee)) - `Rewrap docs/observability.md` ([af8b5c1](fil-forge/piri-signing-service@af8b5c1)) - fil-forge/piri-signing-service#22 ([678c9c1](fil-forge/piri-signing-service@678c9c1))
Sets every staging pin in `terraform/envs/staging/apps/terraform.tfvars` to what `terraform/envs/dev/apps/terraform.tfvars` pins at d0da888. Merging this is the promotion: Check and deploy applies `terraform/envs/staging/apps` on the merge commit, with no further confirmation. dev passed its smoke test on that commit in https://github.com/fil-forge/infra-central/actions/runs/38973848798. This pull request is rebuilt each time dev passes its smoke test after a push to `main`, and closed once staging matches dev. To hold a service back, push to `bot/promote-staging` by hand: the bot leaves the branch alone while this pull request is open. ## sprue [`sha256:79eb33d`](https://github.com/orgs/fil-forge/packages/container/sprue/1339893466) → [`sha256:deab582`](https://github.com/orgs/fil-forge/packages/container/sprue/1399561986) What fil-forge/sprue brings over staging's pin, [`9b40689..88ec247`](fil-forge/sprue@9b40689...88ec247): - fil-forge/sprue#130 ([0b76576](fil-forge/sprue@0b76576)) - fil-forge/sprue#131 ([8c7925a](fil-forge/sprue@8c7925a)) - fil-forge/sprue#132 ([a7762f6](fil-forge/sprue@a7762f6)) - fil-forge/sprue#133 ([a5d84ee](fil-forge/sprue@a5d84ee)) - fil-forge/sprue#137 ([98d22cc](fil-forge/sprue@98d22cc)) - fil-forge/sprue#136 ([9fb2d96](fil-forge/sprue@9fb2d96)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([dbd600d](fil-forge/sprue@dbd600d)) - `Address round 1 review of the observability guidance` ([4188af6](fil-forge/sprue@4188af6)) - fil-forge/sprue#141 ([046155c](fil-forge/sprue@046155c)) - fil-forge/sprue#142 ([80c2ed6](fil-forge/sprue@80c2ed6)) - fil-forge/sprue#145 ([9e5422e](fil-forge/sprue@9e5422e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([a5ee6c7](fil-forge/sprue@a5ee6c7)) - fil-forge/sprue#143 ([3c62127](fil-forge/sprue@3c62127)) - `Rewrap docs/observability.md` ([5c8387a](fil-forge/sprue@5c8387a)) - fil-forge/sprue#140 ([88ec247](fil-forge/sprue@88ec247)) ## hilt [`sha256:5b6d022`](https://github.com/orgs/fil-forge/packages/container/hilt/1384674731) → [`sha256:94be06b`](https://github.com/orgs/fil-forge/packages/container/hilt/1399559133) What fil-forge/hilt brings over staging's pin, [`42dd070..1f0b2c1`](fil-forge/hilt@42dd070...1f0b2c1): - fil-forge/hilt#97 ([b5d3b3e](fil-forge/hilt@b5d3b3e)) - fil-forge/hilt#98 ([1f0b2c1](fil-forge/hilt@1f0b2c1)) ## swarf [`sha256:c364312`](https://github.com/orgs/fil-forge/packages/container/swarf/1361661163) → [`sha256:4807de8`](https://github.com/orgs/fil-forge/packages/container/swarf/1399568771) What fil-forge/swarf brings over staging's pin, [`4b2e2ae..974851a`](fil-forge/swarf@4b2e2ae...974851a): - `Add AGENTS.md with observability guidance` ([88dbb4c](fil-forge/swarf@88dbb4c)) - `Address round 1 review of the observability guidance` ([305021e](fil-forge/swarf@305021e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([7ff3da0](fil-forge/swarf@7ff3da0)) - `Rewrap docs/observability.md` ([72018b4](fil-forge/swarf@72018b4)) - fil-forge/swarf#24 ([974851a](fil-forge/swarf@974851a)) ## delegator [`sha256:30b1757`](https://github.com/orgs/fil-forge/packages/container/delegator/1178963957) → [`sha256:7fb4eb5`](https://github.com/orgs/fil-forge/packages/container/delegator/1399567101) What fil-forge/delegator brings over staging's pin, [`b5552f1..3ae7270`](fil-forge/delegator@b5552f1...3ae7270): - fil-forge/delegator#35 ([8bf4153](fil-forge/delegator@8bf4153)) - fil-forge/delegator#36 ([3dfe8c4](fil-forge/delegator@3dfe8c4)) - fil-forge/delegator#37 ([eb81da2](fil-forge/delegator@eb81da2)) - fil-forge/delegator#38 ([314b6d6](fil-forge/delegator@314b6d6)) - fil-forge/delegator#40 ([e588b27](fil-forge/delegator@e588b27)) - fil-forge/delegator#42 ([9cfc334](fil-forge/delegator@9cfc334)) - fil-forge/delegator#43 ([3d3525c](fil-forge/delegator@3d3525c)) - fil-forge/delegator#45 ([5e4090c](fil-forge/delegator@5e4090c)) - fil-forge/delegator#47 ([22d663d](fil-forge/delegator@22d663d)) - fil-forge/delegator#44 ([984ea6e](fil-forge/delegator@984ea6e)) - fil-forge/delegator#46 ([861f107](fil-forge/delegator@861f107)) - fil-forge/delegator#48 ([5a14d35](fil-forge/delegator@5a14d35)) - fil-forge/delegator#49 ([3fe2769](fil-forge/delegator@3fe2769)) - fil-forge/delegator#52 ([1ee1e5b](fil-forge/delegator@1ee1e5b)) - fil-forge/delegator#51 ([d12f29a](fil-forge/delegator@d12f29a)) - fil-forge/delegator#53 ([8d3b969](fil-forge/delegator@8d3b969)) - fil-forge/delegator#55 ([4d6e592](fil-forge/delegator@4d6e592)) - fil-forge/delegator#56 ([4d37521](fil-forge/delegator@4d37521)) - fil-forge/delegator#59 ([26d9d4e](fil-forge/delegator@26d9d4e)) - fil-forge/delegator#60 ([7eeff65](fil-forge/delegator@7eeff65)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([fe04a4c](fil-forge/delegator@fe04a4c)) - `Address round 1 review of the observability guidance` ([b29fad9](fil-forge/delegator@b29fad9)) - fil-forge/delegator#62 ([aaba55d](fil-forge/delegator@aaba55d)) - fil-forge/delegator#63 ([b3bc907](fil-forge/delegator@b3bc907)) - fil-forge/delegator#64 ([716049e](fil-forge/delegator@716049e)) - fil-forge/delegator#66 ([2c2784d](fil-forge/delegator@2c2784d)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([409ff15](fil-forge/delegator@409ff15)) - `Rewrap docs/observability.md` ([77e9d0a](fil-forge/delegator@77e9d0a)) - fil-forge/delegator#61 ([3ae7270](fil-forge/delegator@3ae7270)) ## signing_service [`sha256:b7ef5f0`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1157634158) → [`sha256:4493534`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1399574922) What fil-forge/piri-signing-service brings over staging's pin, [`c2ac7db..678c9c1`](fil-forge/piri-signing-service@c2ac7db...678c9c1): - fil-forge/piri-signing-service#19 ([8472ad9](fil-forge/piri-signing-service@8472ad9)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([53d3758](fil-forge/piri-signing-service@53d3758)) - `Address round 1 review of the observability guidance` ([950ef02](fil-forge/piri-signing-service@950ef02)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([e07d0ee](fil-forge/piri-signing-service@e07d0ee)) - `Rewrap docs/observability.md` ([af8b5c1](fil-forge/piri-signing-service@af8b5c1)) - fil-forge/piri-signing-service#22 ([678c9c1](fil-forge/piri-signing-service@678c9c1))
Sets every staging pin in `terraform/envs/staging/apps/terraform.tfvars` to what `terraform/envs/dev/apps/terraform.tfvars` pins at a5f29c0. Merging this is the promotion: Check and deploy applies `terraform/envs/staging/apps` on the merge commit, with no further confirmation. dev passed its smoke test on that commit in https://github.com/fil-forge/infra-central/actions/runs/38976031447. This pull request is rebuilt each time dev passes its smoke test after a push to `main`, and closed once staging matches dev. To hold a service back, push to `bot/promote-staging` by hand: the bot leaves the branch alone while this pull request is open. ## sprue [`sha256:79eb33d`](https://github.com/orgs/fil-forge/packages/container/sprue/1339893466) → [`sha256:deab582`](https://github.com/orgs/fil-forge/packages/container/sprue/1399561986) What fil-forge/sprue brings over staging's pin, [`9b40689..88ec247`](fil-forge/sprue@9b40689...88ec247): - fil-forge/sprue#130 ([0b76576](fil-forge/sprue@0b76576)) - fil-forge/sprue#131 ([8c7925a](fil-forge/sprue@8c7925a)) - fil-forge/sprue#132 ([a7762f6](fil-forge/sprue@a7762f6)) - fil-forge/sprue#133 ([a5d84ee](fil-forge/sprue@a5d84ee)) - fil-forge/sprue#137 ([98d22cc](fil-forge/sprue@98d22cc)) - fil-forge/sprue#136 ([9fb2d96](fil-forge/sprue@9fb2d96)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([dbd600d](fil-forge/sprue@dbd600d)) - `Address round 1 review of the observability guidance` ([4188af6](fil-forge/sprue@4188af6)) - fil-forge/sprue#141 ([046155c](fil-forge/sprue@046155c)) - fil-forge/sprue#142 ([80c2ed6](fil-forge/sprue@80c2ed6)) - fil-forge/sprue#145 ([9e5422e](fil-forge/sprue@9e5422e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([a5ee6c7](fil-forge/sprue@a5ee6c7)) - fil-forge/sprue#143 ([3c62127](fil-forge/sprue@3c62127)) - `Rewrap docs/observability.md` ([5c8387a](fil-forge/sprue@5c8387a)) - fil-forge/sprue#140 ([88ec247](fil-forge/sprue@88ec247)) ## hilt [`sha256:5b6d022`](https://github.com/orgs/fil-forge/packages/container/hilt/1384674731) → [`sha256:94be06b`](https://github.com/orgs/fil-forge/packages/container/hilt/1399559133) What fil-forge/hilt brings over staging's pin, [`42dd070..1f0b2c1`](fil-forge/hilt@42dd070...1f0b2c1): - fil-forge/hilt#97 ([b5d3b3e](fil-forge/hilt@b5d3b3e)) - fil-forge/hilt#98 ([1f0b2c1](fil-forge/hilt@1f0b2c1)) ## swarf [`sha256:c364312`](https://github.com/orgs/fil-forge/packages/container/swarf/1361661163) → [`sha256:4807de8`](https://github.com/orgs/fil-forge/packages/container/swarf/1399568771) What fil-forge/swarf brings over staging's pin, [`4b2e2ae..974851a`](fil-forge/swarf@4b2e2ae...974851a): - `Add AGENTS.md with observability guidance` ([88dbb4c](fil-forge/swarf@88dbb4c)) - `Address round 1 review of the observability guidance` ([305021e](fil-forge/swarf@305021e)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([7ff3da0](fil-forge/swarf@7ff3da0)) - `Rewrap docs/observability.md` ([72018b4](fil-forge/swarf@72018b4)) - fil-forge/swarf#24 ([974851a](fil-forge/swarf@974851a)) ## delegator [`sha256:30b1757`](https://github.com/orgs/fil-forge/packages/container/delegator/1178963957) → [`sha256:7fb4eb5`](https://github.com/orgs/fil-forge/packages/container/delegator/1399567101) What fil-forge/delegator brings over staging's pin, [`b5552f1..3ae7270`](fil-forge/delegator@b5552f1...3ae7270): - fil-forge/delegator#35 ([8bf4153](fil-forge/delegator@8bf4153)) - fil-forge/delegator#36 ([3dfe8c4](fil-forge/delegator@3dfe8c4)) - fil-forge/delegator#37 ([eb81da2](fil-forge/delegator@eb81da2)) - fil-forge/delegator#38 ([314b6d6](fil-forge/delegator@314b6d6)) - fil-forge/delegator#40 ([e588b27](fil-forge/delegator@e588b27)) - fil-forge/delegator#42 ([9cfc334](fil-forge/delegator@9cfc334)) - fil-forge/delegator#43 ([3d3525c](fil-forge/delegator@3d3525c)) - fil-forge/delegator#45 ([5e4090c](fil-forge/delegator@5e4090c)) - fil-forge/delegator#47 ([22d663d](fil-forge/delegator@22d663d)) - fil-forge/delegator#44 ([984ea6e](fil-forge/delegator@984ea6e)) - fil-forge/delegator#46 ([861f107](fil-forge/delegator@861f107)) - fil-forge/delegator#48 ([5a14d35](fil-forge/delegator@5a14d35)) - fil-forge/delegator#49 ([3fe2769](fil-forge/delegator@3fe2769)) - fil-forge/delegator#52 ([1ee1e5b](fil-forge/delegator@1ee1e5b)) - fil-forge/delegator#51 ([d12f29a](fil-forge/delegator@d12f29a)) - fil-forge/delegator#53 ([8d3b969](fil-forge/delegator@8d3b969)) - fil-forge/delegator#55 ([4d6e592](fil-forge/delegator@4d6e592)) - fil-forge/delegator#56 ([4d37521](fil-forge/delegator@4d37521)) - fil-forge/delegator#59 ([26d9d4e](fil-forge/delegator@26d9d4e)) - fil-forge/delegator#60 ([7eeff65](fil-forge/delegator@7eeff65)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([fe04a4c](fil-forge/delegator@fe04a4c)) - `Address round 1 review of the observability guidance` ([b29fad9](fil-forge/delegator@b29fad9)) - fil-forge/delegator#62 ([aaba55d](fil-forge/delegator@aaba55d)) - fil-forge/delegator#63 ([b3bc907](fil-forge/delegator@b3bc907)) - fil-forge/delegator#64 ([716049e](fil-forge/delegator@716049e)) - fil-forge/delegator#66 ([2c2784d](fil-forge/delegator@2c2784d)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([409ff15](fil-forge/delegator@409ff15)) - `Rewrap docs/observability.md` ([77e9d0a](fil-forge/delegator@77e9d0a)) - fil-forge/delegator#61 ([3ae7270](fil-forge/delegator@3ae7270)) ## signing_service [`sha256:b7ef5f0`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1157634158) → [`sha256:4493534`](https://github.com/orgs/fil-forge/packages/container/piri-signing-service/1399574922) What fil-forge/piri-signing-service brings over staging's pin, [`c2ac7db..678c9c1`](fil-forge/piri-signing-service@c2ac7db...678c9c1): - fil-forge/piri-signing-service#19 ([8472ad9](fil-forge/piri-signing-service@8472ad9)) - `AGENTS.md: add telemetry, alerts and dashboards to the definition of done` ([53d3758](fil-forge/piri-signing-service@53d3758)) - `Address round 1 review of the observability guidance` ([950ef02](fil-forge/piri-signing-service@950ef02)) - `Keep the always-loaded guidance short, and move the how-to to docs` ([e07d0ee](fil-forge/piri-signing-service@e07d0ee)) - `Rewrap docs/observability.md` ([af8b5c1](fil-forge/piri-signing-service@af8b5c1)) - fil-forge/piri-signing-service#22 ([678c9c1](fil-forge/piri-signing-service@678c9c1))
Written by Claude.
Merge after fil-forge/infra-central#253. This PR's
docs/observability.mdpoints to the "Adding an alert or panel" section that PR adds.This adds a short Observability section to
AGENTS.md. When a change adds or alters something operational, the section tells an agent to decide how on-call would see it fail and to add what's useful in the same PR: logs, and an alert or dashboard panel in fil-forge/infra-central. It also says that when a feature's health is best read as a number, the PR description should name the metric and FIL-1152. It also says to skip this for changes nothing operational depends on.The section is kept short because
AGENTS.mdloads into every session. That follows bajtos's review of fil-forge/infra-nodes#174. The how-to lives in a newdocs/observability.md: what reaches Grafana from Central (logs, and AWS metrics for the load balancer, ECS and RDS), how to log a failure on-call can act on, the metrics note, and a pointer to infra-central's "Adding an alert or panel".This PR no longer adds a
CLAUDE.md. Claude Code readsAGENTS.mditself from v2.1.277 (docs).Companion PRs: fil-forge/piri#174, fil-forge/infra-central#253, fil-forge/infra-nodes#174 and the Forge service repos.
Checks
git diff --check.✴️
🤖 Generated with Claude Code
https://claude.ai/code/session_01JCF8gyb8jNRmHmWqQmVhRJ
Generated by Claude Code