Skip to content

Commit d6d32e1

Browse files
authored
Merge branch 'master' into tony/flusher-logger-tool
2 parents 361adb4 + bfda068 commit d6d32e1

File tree

1,757 files changed

+26194
-13964
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,757 files changed

+26194
-13964
lines changed

.envrc

Lines changed: 17 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,7 @@ if [ "$(uname -s)" == "Darwin" ] && [ ! -f "/Library/Developer/CommandLineTools/
1616
return 1
1717
fi
1818

19-
SENTRY_ROOT="$(
20-
cd "$(dirname "${BASH_SOURCE[0]}")"
21-
pwd -P
22-
)"
23-
24-
source "${SENTRY_ROOT}/scripts/lib.sh"
19+
source "scripts/lib.sh"
2520

2621
bold="$(tput bold)"
2722
red="$(tput setaf 1)"
@@ -106,9 +101,9 @@ export SENTRY_UI_HOT_RELOAD=1
106101

107102
### You can override the exported variables with a .env file
108103
# All exports should happen before here unless they're safeguarded (see devenv error reporting below)
109-
if [ -f "${SENTRY_ROOT}/.env" ]; then
110-
info "Loading variables from ${SENTRY_ROOT}/.env"
111-
dotenv "${SENTRY_ROOT}/.env"
104+
if [ -f ".env" ]; then
105+
info "Loading variables from .env"
106+
dotenv
112107
fi
113108

114109
## Notify of reporting to Sentry
@@ -125,24 +120,19 @@ else
125120
export SENTRY_DEVSERVICES_DSN=https://23670f54c6254bfd9b7de106637808e9@o1.ingest.sentry.io/1492057
126121
fi
127122

123+
# for deduplicating
124+
PATH_rm "/opt/homebrew/bin"
128125

129-
### System ###
130-
131-
for pkg in \
132-
make \
133-
docker \
134-
chromedriver \
135-
openssl; do
136-
if ! require "$pkg"; then
137-
die "You seem to be missing the system dependency: ${pkg}
138-
Please install homebrew, and run brew bundle."
139-
fi
140-
done
126+
# homebrew must not be searched first, we need venv and devenv to take precedence
127+
PATH_add "/opt/homebrew/bin"
141128

142129
### Python ###
143130

144-
if [ -f .venv/bin/devenv ]; then
145-
DEVENV=.venv/bin/devenv
131+
export VIRTUAL_ENV="${PWD}/.venv"
132+
PATH_add "${VIRTUAL_ENV}/bin"
133+
134+
if [ -f "${VIRTUAL_ENV}/bin/devenv" ]; then
135+
DEVENV="${VIRTUAL_ENV}/bin/devenv"
146136
else
147137
DEVENV=devenv
148138
fi
@@ -156,9 +146,6 @@ https://github.com/getsentry/devenv#install
156146
'
157147
fi
158148

159-
PATH_add .venv/bin
160-
export VIRTUAL_ENV="$PWD/.venv"
161-
162149
if ! require sentry; then
163150
warn "Your virtualenv is activated, but sentry doesn't seem to be installed."
164151
commands_to_run+=("devenv sync")
@@ -177,25 +164,23 @@ if ! require pre-commit; then
177164
fi
178165

179166
### dotagents ###
180-
if [ -f "${SENTRY_ROOT}/agents.toml" ] && [ ! -d "${SENTRY_ROOT}/.agents/skills/commit" ]; then
167+
168+
if [ -f "agents.toml" ] && [ ! -d ".agents/skills/commit" ]; then
181169
warn "Agent skills not installed."
182170
commands_to_run+=("devenv sync")
183171
fi
184172

185-
python3 -m tools.docker_memory_check
186-
187173
### Node ###
188174

189-
# not needed for getsentry
190-
if [ "${PWD##*/}" = "sentry" ]; then
175+
if [ -f ".node-version" ]; then
191176
debug "Checking node..."
192177

193178
if [ "${SENTRY_DEVENV_SKIP_FRONTEND:-}" != "1" ]; then
194179
if ! require node; then
195180
die "You don't seem to have node installed. Please run devenv sync."
196181
fi
197182

198-
read -r node_version < .node-version
183+
read -r node_version < ".node-version"
199184
if [ "v${node_version}" != "$(node --version)" ]; then
200185
die "Unexpected $(command -v node) version. Please run devenv sync."
201186
fi

.github/CODEOWNERS

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -512,18 +512,21 @@ tests/sentry/api/endpoints/test_organization_attribute_mappings.py @get
512512
/src/sentry/tasks/ai_agent_monitoring.py @getsentry/telemetry-experience
513513
/tests/sentry/tasks/test_ai_agent_monitoring.py @getsentry/telemetry-experience
514514
/static/app/actionCreators/metrics.tsx @getsentry/telemetry-experience
515-
/static/app/data/platformCategories.tsx @getsentry/telemetry-experience
516-
/static/app/data/platformPickerCategories.tsx @getsentry/telemetry-experience
517-
/static/app/data/platforms.tsx @getsentry/telemetry-experience
518-
/static/app/gettingStartedDocs/ @getsentry/telemetry-experience
519-
/static/app/types/project.tsx @getsentry/telemetry-experience
520515
/static/app/views/settings/dynamicSampling/ @getsentry/telemetry-experience
521516
/static/app/views/insights/pages/agents/ @getsentry/telemetry-experience
522517
/static/app/views/insights/pages/mcp/ @getsentry/telemetry-experience
523518
/static/app/views/insights/pages/platform/ @getsentry/telemetry-experience
524519
/static/app/views/insights/pages/conversations/ @getsentry/telemetry-experience
525520
## End of Telemetry Experience
526521

522+
## Value Discovery
523+
/static/app/data/platformCategories.tsx @getsentry/value-discovery
524+
/static/app/data/platformPickerCategories.tsx @getsentry/value-discovery
525+
/static/app/data/platforms.tsx @getsentry/value-discovery
526+
/static/app/gettingStartedDocs/ @getsentry/value-discovery
527+
/static/app/types/project.tsx @getsentry/value-discovery
528+
## End of Value Discovery
529+
527530
## ML & AI
528531
/static/app/components/events/autofix/ @getsentry/machine-learning-ai
529532
/static/app/views/seerExplorer/ @getsentry/machine-learning-ai
@@ -626,6 +629,13 @@ tests/sentry/api/endpoints/test_organization_attribute_mappings.py @get
626629
/tests/snuba/search/ @getsentry/issue-workflow
627630
## End of Issues
628631

632+
## Onboarding (Value Discovery)
633+
# These must come after /src/sentry/api/endpoints/ catch-all above
634+
/src/sentry/api/endpoints/organization_onboarding_tasks.py @getsentry/value-discovery
635+
/src/sentry/api/endpoints/organization_onboarding_continuation_email.py @getsentry/value-discovery
636+
/src/sentry/api/endpoints/project_create_sample.py @getsentry/value-discovery
637+
## End of Onboarding
638+
629639
## AI Conversations (Telemetry Experience)
630640
# These must come after /src/sentry/api/endpoints/ catch-all above
631641
/src/sentry/api/endpoints/organization_ai_conversation*.py @getsentry/telemetry-experience

.github/workflows/backend-with-coverage.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ on:
77
schedule:
88
- cron: '0,30 * * * *'
99

10+
env:
11+
SENTRY_SKIP_SELENIUM_PLUGIN: '1'
12+
1013
jobs:
1114
# Skip generating coverage if already exists in GCS
1215
check-existing-coverage:

.github/workflows/backend.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ concurrency:
1818
env:
1919
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 3
2020
SNUBA_NO_WORKERS: 1
21+
SENTRY_SKIP_SELENIUM_PLUGIN: '1'
2122

2223
jobs:
2324
files-changed:

.github/workflows/getsentry-dispatch.yml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,23 +56,25 @@ jobs:
5656
private_key: ${{ secrets.SENTRY_INTERNAL_APP_PRIVATE_KEY }}
5757

5858
- name: Wait for PR merge commit
59-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
59+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
6060
id: mergecommit
6161
with:
6262
github-token: ${{ steps.getsentry.outputs.token }}
6363
script: |
64-
require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/wait-for-merge-commit`).waitForMergeCommit({
64+
const { waitForMergeCommit } = await import(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/wait-for-merge-commit.js`);
65+
await waitForMergeCommit({
6566
github,
6667
context,
6768
core,
6869
});
6970
7071
- name: Dispatch getsentry tests
71-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
72+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
7273
with:
7374
github-token: ${{ steps.getsentry.outputs.token }}
7475
script: |
75-
require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/getsentry-dispatch`).dispatch({
76+
const { dispatch } = await import(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/getsentry-dispatch.js`);
77+
await dispatch({
7678
github,
7779
context,
7880
core,

.github/workflows/meta-deploys-detect-change-type.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,11 @@ jobs:
2323
filters: .github/file-filters.yml
2424

2525
- name: Create GitHub job
26-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
26+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
2727
with:
2828
script: |
29-
require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/deploy`).updateChangeType({
29+
const { updateChangeType } = await import(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/deploy.js`);
30+
await updateChangeType({
3031
github,
3132
context,
3233
fileChanges: ${{ toJson(steps.changes.outputs) }}

.github/workflows/sentry-pull-request-bot.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
steps:
2424
- name: Check getsentry membership
2525
id: org
26-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
26+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
2727
with:
2828
script: |
2929
try {
@@ -45,24 +45,25 @@ jobs:
4545
private_key: ${{ secrets.SENTRY_INTERNAL_APP_PRIVATE_KEY }}
4646

4747
- name: Wait for PR merge commit
48-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
48+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
4949
id: mergecommit
5050
with:
5151
github-token: ${{ steps.getsentry.outputs.token }}
5252
script: |
53-
require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/wait-for-merge-commit`).waitForMergeCommit({
53+
const { waitForMergeCommit } = await import(`${process.env.GITHUB_WORKSPACE}/.github/workflows/scripts/wait-for-merge-commit.js`);
54+
await waitForMergeCommit({
5455
github,
5556
context,
5657
core,
5758
});
5859
5960
- name: Dispatch getsentry tests
6061
if: steps.org.outputs.result == 'true'
61-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
62+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
6263
with:
6364
github-token: ${{ steps.getsentry.outputs.token }}
6465
script: |
65-
github.rest.actions.createWorkflowDispatch({
66+
await github.rest.actions.createWorkflowDispatch({
6667
owner: 'getsentry',
6768
repo: 'getsentry',
6869
workflow_id: 'acceptance.yml',

.github/workflows/sync-labels.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,13 +42,13 @@ jobs:
4242
2>&1 | tee sync-report.txt
4343
4444
- name: Read sync output into variable
45-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
45+
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
4646
if: github.event_name == 'pull_request'
4747
id: github-label-sync
4848
with:
4949
script: |
50-
const fs = require('fs');
51-
return fs.readFileSync('sync-report.txt','utf8').toString();
50+
const { readFileSync } = await import('node:fs');
51+
return readFileSync('sync-report.txt', 'utf8');
5252
result-encoding: string
5353

5454
- name: Find previous dry-run comment

.github/workflows/warden.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ jobs:
1313
env:
1414
WARDEN_ANTHROPIC_API_KEY: ${{ secrets.WARDEN_ANTHROPIC_API_KEY }}
1515
WARDEN_MODEL: ${{ secrets.WARDEN_MODEL }}
16+
WARDEN_SENTRY_DSN: ${{ secrets.WARDEN_SENTRY_DSN }}
1617
steps:
1718
- uses: actions/checkout@v4
1819

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ package-lock.json
6464
yarn.lock
6565
.claude/settings.local.json
6666
.claude/plans
67+
.claude/worktrees
6768
.warden/
6869
# Auto-updated agent skill dependencies — ignore to avoid dirty state
6970
.agents/.gitignore

0 commit comments

Comments
 (0)