Skip to content

Add optional description field to App configs#91

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/add-description-arg-for-app-types
Draft

Add optional description field to App configs#91
Copilot wants to merge 3 commits intomainfrom
copilot/add-description-arg-for-app-types

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 18, 2026

Adds an optional description?: pulumi.Input<string> field to all three mega-component arg interfaces, enabling resource descriptions to be passed as plain strings, Pulumi inputs, or outputs.

Changes

  • src/packages/gcp/apps.ts — Added description?: pulumi.Input<string> to StaticWebAppArgs, CloudRunServiceArgs, and FullStackAppArgs
  • tests/integration/ — Added a description field test block to each existing integration test file (static-webapp, cloudrun-with-db, fullstack-app), replacing the previous standalone unit test file

Usage

new FullStackApp('my-app', {
  description: pulumi.interpolate`${env}-my-app`, // pulumi.Output<string> ✓
  description: config.get('appDescription'),      // pulumi.Input<string> ✓
  description: 'my app',                          // string ✓
  // omitted entirely                             // also valid ✓
  loadBalancer: { ... },
  frontend: { ... },
  backend: { ... },
});

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • api.pulumi.com
    • Triggering command: /usr/local/bin/pulumi pulumi stack select --stack test-1773864200033 --non-interactive (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Add description arg for each of the app types</issue_title>
<issue_description>For each of the mega component apps listed below they should contain an optional description argument that could be useful in a few cases and should accept a pulumi.Input. Commented as "An optional description of this resource"

StaticWebApp -> StaticWebAppArgs
CloudRunService -> CloudRunServiceArgs
FullStackApp -> FullStackAppArgs
</issue_description>

<agent_instructions>This should only modify the src/packages/gcp/apps.ts script and tests. Tests should be updated to verify that description can accept a pulumi.Input or pulumi.Output</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…gs, and FullStackAppArgs

Co-authored-by: Srayman <42743016+Srayman@users.noreply.github.com>
Copilot AI changed the title [WIP] Add description arg for each of the app types Add optional description field to StaticWebAppArgs, CloudRunServiceArgs, and FullStackAppArgs Mar 18, 2026
Copilot AI requested a review from Srayman March 18, 2026 19:58
…ve unit test file

Co-authored-by: Srayman <42743016+Srayman@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@Srayman Srayman changed the title Add optional description field to StaticWebAppArgs, CloudRunServiceArgs, and FullStackAppArgs Add optional description field to App configs Mar 18, 2026
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.

Add description arg for each of the app types

2 participants