Skip to content

fix(web): soften reviewed/durable Studio copy and pad footer - #1704

Merged
kk-agent merged 3 commits into
mainfrom
cursor/studio-soften-overclaim-9093
Sep 8, 2026
Merged

kk-agent merged 3 commits into
mainfrom
cursor/studio-soften-overclaim-9093

Conversation

@groupthinking

@groupthinking groupthinking commented Sep 8, 2026 •

Copy link
Copy Markdown
Owner

Canonical issue

Hayden dogfood residual G after E/#1703 and gate-UI #1717 (ede5b2a1). Rebased onto current main. Unparked after G.A.T.E. receipt chip landed.

Outcome

Softer product copy (STUDIO_PRODUCT_TAGLINE) and Studio pb-20 footer pad. No “durable Workflow DevKit” comment on the retired Studio surface.

Preserved from main: G.A.T.E. studio-gate-receipt chip, review cards, export toast, player, and claim-soft (no “Deploy completed”).

Lockfile sync so Docker npm ci --workspace=apps/web --production --legacy-peer-deps matches apps/web OpenTelemetry ^0.222.0.

Scope

  • Included: tagline + footer pad + comment cleanup + package-lock.json OTEL 0.221→0.222 sync
  • Explicitly excluded: G.A.T.E. redesign, fake events, ads, any removal of studio-gate-receipt, weakening/skipping Trivy

Risk

  • Risk level: low
  • Failure mode: footer pad or tagline regresses after a later Studio layout change
  • Rollback: revert this PR

Verification

Head SHA: 043ece5e7 on ede5b2a1

  • Unique vs main: 2 G soften commits + 1 lockfile sync
  • studio-gate-receipt still present in OneLoopStudio.tsx
  • Local npm ci --workspace=apps/web --production --legacy-peer-deps succeeds
  • Core CI green on 043ece5e7: build, test, test-frontend, lint-frontend, guards
  • Trivy SUCCESS on 043ece5e7 (image build + scan)

QA on pBsT6v-ciO8

  1. Open /studio — footer/tagline must not say “reviewed actions” / “durable workflows”.
  2. Studio main must have bottom pad so the footer does not cover actions.
  3. After Attempt deploy, studio-gate-receipt chip still shows PASS|HOLD|REJECT|ESCALATE + receipt id/hash.
  4. Do not see “Deploy completed”.

Loop: merge-when-green. Trivy is green. Not merging from this agent. Studio residuals pause after merge.

Open in Web Open in Cursor 

@vercel

vercel Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Sep 8, 2026 8:17pm UTC

@github-actions github-actions Bot added javascript Pull requests that update javascript code tests labels Sep 8, 2026
@coderabbitai

coderabbitai Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 33fb138c-1e59-4434-9023-ae5529c6d224


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 043ece5.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails
npm/@opentelemetry/exporter-trace-otlp-http 0.222.0 🟢 7.3
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Dependency-Update-Tool🟢 10update tool detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
License🟢 10license file detected
Branch-Protection🟢 4branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 19 existing vulnerabilities detected
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 40 contributing companies or organizations
npm/@opentelemetry/otlp-exporter-base 0.222.0 🟢 7.3
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Dependency-Update-Tool🟢 10update tool detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
License🟢 10license file detected
Branch-Protection🟢 4branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 19 existing vulnerabilities detected
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 40 contributing companies or organizations
npm/@opentelemetry/otlp-transformer 0.222.0 🟢 7.3
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Dependency-Update-Tool🟢 10update tool detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
License🟢 10license file detected
Branch-Protection🟢 4branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 19 existing vulnerabilities detected
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 40 contributing companies or organizations
npm/@opentelemetry/sdk-logs 0.222.0 🟢 7.3
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Dependency-Update-Tool🟢 10update tool detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
License🟢 10license file detected
Branch-Protection🟢 4branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 19 existing vulnerabilities detected
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 40 contributing companies or organizations
npm/@opentelemetry/sdk-metrics 2.11.0 🟢 7.3
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 6 issue activity found in the last 90 days -- score normalized to 10
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Dependency-Update-Tool🟢 10update tool detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
License🟢 10license file detected
Branch-Protection🟢 4branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 19 existing vulnerabilities detected
Signed-Releases⚠️ 0Project has not signed or included provenance with any releases.
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 40 contributing companies or organizations

Scanned Files

  • package-lock.json

Copilot AI previously approved these changes Sep 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approved

The changes are low-risk copy/layout adjustments with targeted test coverage; only a minor comment wording nit remains.

Pull request overview

Updates Studio/marketing copy to avoid “reviewed actions” / “durable workflows” overclaims, and ensures Studio results aren’t covered by the sticky action bar.

Changes:

  • Introduces STUDIO_PRODUCT_TAGLINE and uses it in the marketing footer.
  • Softens “durable workflow” phrasing in the (retired) VideoWorkflowStudio copy paths.
  • Adds bottom padding (pb-20) to Studio main content and a data-testid hook for verification.
File summaries
File Description
apps/web/src/lib/studio-pipeline-status.ts Adds STUDIO_PRODUCT_TAGLINE constant used by the footer.
apps/web/src/lib/tests/studio-pipeline-status.test.ts Adds a regression test ensuring overclaim copy is removed and Studio main is padded.
apps/web/src/components/VideoWorkflowStudio.tsx Rewords user-facing messages to remove “durable workflow” language.
apps/web/src/components/OneLoopStudio.tsx Adds pb-20 to prevent sticky footer/action bar overlap; adds data-testid="studio-main".
apps/web/src/components/Footer.tsx Replaces footer overclaim copy with STUDIO_PRODUCT_TAGLINE.
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

setActionsBusy(true);
setWorkflowActions(null);
setActionMessage('Starting durable video-to-actions workflow…');
setActionMessage('Starting video-to-actions run…');
@cursor
cursor Bot force-pushed the cursor/studio-review-action-cards-9093 branch from 3a663f9 to 719bedc Compare September 8, 2026 17:41
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from 638c595 to 63fd9f5 Compare September 8, 2026 17:41
@cursor
cursor Bot force-pushed the cursor/studio-review-action-cards-9093 branch from 719bedc to e2f0c22 Compare September 8, 2026 17:48
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from 63fd9f5 to 9489689 Compare September 8, 2026 17:48
@cursor
cursor Bot force-pushed the cursor/studio-review-action-cards-9093 branch from e2f0c22 to 4a51bf4 Compare September 8, 2026 17:55
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from 9489689 to a29d026 Compare September 8, 2026 17:55
@cursor
cursor Bot force-pushed the cursor/studio-review-action-cards-9093 branch from 4a51bf4 to 12e14d5 Compare September 8, 2026 18:26
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from a29d026 to 0eadab4 Compare September 8, 2026 18:26
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from 0eadab4 to a18e6eb Compare September 8, 2026 18:35
@cursor
cursor Bot changed the base branch from cursor/studio-review-action-cards-9093 to main September 8, 2026 18:37
@cursor
cursor Bot dismissed Copilot’s stale review September 8, 2026 18:37

The base branch was changed.

@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from a18e6eb to b583fc9 Compare September 8, 2026 18:44
@kk-agent

kk-agent commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

PARKED by UVAI Loop (CoS): G.A.T.E. verdict/receipt UI cut takes sole-writer priority after AXIOM spot-check FAIL (B). Will resume #1704 merge-when-green after gate-UI lands.

cursoragent and others added 2 commits September 8, 2026 19:03
Replace overclaim taglines with Video Pack language and add bottom padding so the sticky action bar does not cover results.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
Keep Studio terminology consistent with the tagline change so comments do not reintroduce the overclaim.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
@cursor
cursor Bot force-pushed the cursor/studio-soften-overclaim-9093 branch from b583fc9 to 5a1e7c5 Compare September 8, 2026 19:04
@cursor

cursor Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

UNPARKED: rebased onto current main (ede5b2a1, gate-UI #1717).

Preserved: studio-gate-receipt chip + review cards + export toast + player + claim-soft. Unique delta remains G only (tagline + pb-20 footer pad + durable-copy soften).

Head: 5a1e7c5f0. Waiting on core CI (Trivy OK) then MERGEABLE for Loop squash-merge.

@cursor

cursor Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Ready for Loop squash-merge.

Studio residuals pause after squash-merge.

apps/web declares exporter-trace-otlp-http ^0.222.0; the lockfile still
resolved 0.221.0, so Trivy image build failed before the scan ran.

Co-authored-by: Hayden <groupthinking@users.noreply.github.com>
@cursor

cursor Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Lockfile sync pushed to clear inherited Trivy image-build fail.

  • Hypothesis confirmed: apps/web declares @opentelemetry/exporter-trace-otlp-http@^0.222.0; lockfile still had ^0.221.0.
  • Commit 043ece5e7 updates package-lock.json only (0.221 → 0.222 + missing OTEL entries).
  • Local proof: npm ci --workspace=apps/web --production --legacy-peer-deps now succeeds.
  • Waiting on CI for trivy SUCCESS on this SHA. Loop: merge-when-green. Not merging from this agent.

@cursor

cursor Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Trivy is green. Loop can merge-when-green.

Not merging from this agent. Studio residuals pause after squash-merge.

@kk-agent
kk-agent merged commit 8ff62ec into main Sep 8, 2026
28 of 29 checks passed
@kk-agent
kk-agent deleted the cursor/studio-soften-overclaim-9093 branch September 8, 2026 20:24
@linear-code

linear-code Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

GRV-508

This branch was successfully deployed

1 active deployment
Preview — 043ece5e Deployed Sep 8, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update javascript code tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants