Repository navigation
feat(sns-discord): post CloudWatch alarms and AWS Budgets notifications in Korean - #210
Merged
Merged
Conversation
Build the message from the alarm's Trigger and the datapoints in NewStateReason instead of passing AWS's English reason through, and label the chart's threshold line 기준. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NjZkQtRoTFu44uBp8tgofc
…elds The embed listed every key of the SNS message, ARNs and actions included, in AWS's English. It now carries the alarm's own description, which the chart no longer clears. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NjZkQtRoTFu44uBp8tgofc
Budgets publish plain English text, which was posted as a code block under an empty line. Read its Budget Name, Alert Type, Alert Threshold and amount lines into one Korean sentence; anything else that is not JSON is still dumped as before. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NjZkQtRoTFu44uBp8tgofc
The statistic names no longer carry 이/가 themselves. hgtk ships a pure Python wheel, so the deploy's --only-binary install takes it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NjZkQtRoTFu44uBp8tgofc
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
1 participant
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.
Alarm and budget posts in Discord now read in Korean, with times in KST. Today they pass AWS's English
NewStateReasonthrough, and the embed lists every raw key of the SNS message.The CloudFront OK from 10:46 KST today becomes:
A budget becomes
[예산 lambda] 실제 비용이 알림 기준(> $0.01)을 넘었습니다. 실제 $0.02, 예산 $1.00.Trigger(statistic, period, comparison, threshold, evaluation periods). The datapoints are not a structured field in the SNS message, so they are read out ofNewStateReason; if that text changes shape, the datapoints drop out and the rest stays.AlarmDescription(none of the alarms set one yet) instead of the raw ARNs and actions. The chart no longer clears it.Budget Name:and so on). Any other non-JSON message is dumped as before.hgtk(pure Python wheel, no dependencies) attaches 이/가 to the statistic name, in a newsns-discorddependency group.Review commit by commit. Since
uv.lockchanges, merging redeploys all three functions, not onlysns-discord; after it lands I send a_Testmessage throughCloudWatch_Alarms_Topic.🤖 Generated with Claude Code
https://claude.ai/code/session_01NjZkQtRoTFu44uBp8tgofc