Skip to content

chore: lint shared/ folder - #1169

Merged
raivieiraadriano92 merged 3 commits into
mainfrom
raivieiraadriano92/lint-shared-folder
Jul 30, 2026
Merged

chore: lint shared/ folder#1169
raivieiraadriano92 merged 3 commits into
mainfrom
raivieiraadriano92/lint-shared-folder

Conversation

@raivieiraadriano92

Copy link
Copy Markdown
Collaborator

Cherry-picks the chore: lint shared/ folder commit (339c1ea9) from #1168 onto main as a standalone change.

Applies ESLint to the shared/ folder: adds the lint config wiring in eslint.config.js/package.json and applies the resulting autofixes across shared/agent-core, shared/ip-classification, shared/url, and a few src/ consumers.

Notably renames the exported constant APP_HARNESS_ENVIRONMENT_PROMPTappHarnessEnvironmentPrompt and updates all call sites.

Conflict resolution

One conflict in src/ai/prompt.test.ts: main had added a widgetRegistry import alongside the old APP_HARNESS_ENVIRONMENT_PROMPT name. Resolved by keeping the widgetRegistry import and using the renamed appHarnessEnvironmentPrompt. All other references to the old name across the tree were updated by the commit.

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

Semgrep Security Scan

No security issues found.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

Preview environment destroyed 🧹

Stack preview-pr-1169 and its Cloudflare subdomain have been cleaned up.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

PR Metrics

Metric Value
Lines changed (prod code) +256 / -119
JS bundle size (gzipped) 🟢 585.4 KB → 585.2 KB (-161 B, -0.0%)
Test coverage 🟢 81.00% → 81.00% (+0.0%)
Performance (preview) Preview not ready — Render deploy may have timed out
Accessibility
Best Practices
SEO

Updated Thu, 30 Jul 2026 13:11:26 GMT · run #2514

github-actions[bot]

This comment was marked as outdated.

github-actions[bot]

This comment was marked as outdated.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔭 thunder-deep-review (advisory)

Reviewed the diff — no issues to report. ✅ Never approves, never requests changes, never gates merge.
head: 8f6cf0d4d879 · mode: deep · deferred 0 item(s) already reported by other bots (best-effort dedup)

@raivieiraadriano92
raivieiraadriano92 merged commit e9678f1 into main Jul 30, 2026
31 checks passed
@raivieiraadriano92
raivieiraadriano92 deleted the raivieiraadriano92/lint-shared-folder branch July 30, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant