Skip to content

Conversation

Copy link

Copilot AI commented Jan 22, 2026

Purpose

Empty markdown cell rendered in HTTP notebook when invoking "Try it" from a resource, showing placeholder text "Empty markdown cell, double-click or press enter to edit."

Goals

Eliminate the empty markdown cell while preserving existing functionality for service-level try-it invocations.

Approach

The Handlebars template conditionally hides method/path headers in resource mode using {{#unless isResourceMode}}, but only wrapped the content, leaving the markdown comment delimiters /* */ to render as an empty cell.

Changed: Moved the conditional to wrap the entire markdown block instead of just its content.

{{#each paths}}
{{#each this}}
-/*
-{{#unless ../../isResourceMode}}#### {{uppercase @key}} {{@../key}}{{/unless}}
+{{#unless ../../isResourceMode}}/*
+#### {{uppercase @key}} {{@../key}}

{{#if parameters}}
  ... parameter docs ...
{{/if}}
*/
+{{/unless}}
###

Before (resource mode):

/*
### Try Resource: 'POST /toxml'
*/

/*

*/  ← Empty markdown cell
###
POST http://localhost:9090/toxml

After (resource mode):

/*
### Try Resource: 'POST /toxml'
*/

###  ← Block removed entirely
POST http://localhost:9090/toxml

Service mode behavior unchanged—method headers still render correctly.

UI Component Development

  • N/A - Template change only

Manage Icons

  • N/A - No icon changes

User stories

Users invoking "Try it" from a resource no longer see an empty markdown cell disrupting the HTTP notebook layout.

Release note

Fixed empty markdown cell appearing in HTTP notebook when testing individual resources via "Try it" feature.

Documentation

N/A - UI fix with no new user-facing functionality.

Training

N/A

Certification

N/A - Bug fix with no certification impact.

Marketing

N/A

Automation tests

  • Unit tests: Template compilation verified with Handlebars
  • Integration tests: N/A - UI rendering change

Security checks

  • Followed secure coding standards: yes
  • Ran FindSecurityBugs plugin: N/A - template string change only
  • Confirmed no secrets committed: yes

Samples

N/A

Related PRs

None

Migrations (if applicable)

N/A

Test environment

Node.js with Handlebars 4.x - template compilation verified

Learning

Handlebars conditional block scope—conditionals that only hide content still render surrounding markup. Solution: wrap entire block structure in conditional.

Original prompt

This section details on the original issue you should resolve

<issue_title>Additional markdown cell rendered in Resource try it</issue_title>
<issue_description>### Description

$subject

Image

Steps to Reproduce

Run Try it from resouce
we could see an additional markdown cell rendered

Version

0.9.825040911

Environment Details (with versions)

No response</issue_description>

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.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@axewilledge axewilledge marked this pull request as ready for review January 22, 2026 06:16
Copilot AI changed the title [WIP] Fix additional markdown cell rendered in Try it resource Remove empty markdown cell in resource try-it view Jan 22, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 22, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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

Copilot AI requested a review from axewilledge January 22, 2026 06:17
@axewilledge axewilledge changed the base branch from main to bi-1.6.x January 22, 2026 06:23
@axewilledge axewilledge changed the base branch from bi-1.6.x to main January 22, 2026 06:23
@axewilledge axewilledge reopened this Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Additional markdown cell rendered in Resource try it

3 participants