feat: Featured Snippets, search and filtering for Community page by code-snippets-bot · Pull Request #383 · codesnippetspro/code-snippets

code-snippets-bot · 2026-05-05T13:51:46Z

Summary

Community Snippets page now shows featured snippets on initial load and supports server-side filtering with category, type, and status dropdowns. Filter options come from the full query result via independent facets.

Featured Snippets

Loads featured snippets when no search query active
Paginated via server-side page/per_page params
Featured heading visible above results

Server-Side Filters

category, type, status params sent to cloud API (ID-based)
Filter dropdowns built from available_filters response header (X-WP-Filters)
Independent facets: active filter dimension keeps all options, other dimensions narrow
Filters reset to page 1 on change

New Endpoints Proxied

GET /cloud/types (languages from cloud API, 1-day transient)
GET /cloud/categories (categories from cloud API, 1-day transient)

Cloud Snippet Model

Added language field to Cloud_Snippet (extracted from API response language.name)
Cloud_Snippets picks up available_filters from normalized API response

UI

Type dropdown added to search filters (alongside category and status)
Spinner replaces search button label during loading
Consistent heading and filters visible on empty results

Test plan

Featured snippets load on community page with no search
Type filter narrows results, categories/statuses adapt
Screen options per_page respected
Pagination works with filters active
Search clears featured and applies filters

Add get_featured_snippets() to Cloud_API that fetches from the cloud /api/v1/featured endpoint with transient caching derived from the cached_until response field. Add /cloud/featured REST route to expose featured snippets to the frontend.

Load featured snippets on the Community Cloud page when no search query is active. The frontend context fetches from the /cloud/featured REST endpoint on mount and displays results under a "Featured Snippets" heading. When the user performs a search, normal search behaviour takes over. Add PHPUnit tests covering transient caching, cache-hit bypass, and graceful fallback on HTTP errors.

Add 3 additional PHPUnit tests: TTL respects cached_until, missing data key returns empty result, and transient stores Cloud_Snippets instance. Add Playwright E2E spec for community cloud page covering page load, featured heading display, and search override behavior.

…d tests

louiswol94

This PR is clean!

The only suggestion I can make is to mock the responses via Playwright page.route() + route.fulfill() with fixed JSON and X-WP-Total / X-WP-TotalPages headers. That avoids depending on a live cloud/API, removes flakiness from timeouts and env differences, and lets us assert the featured heading, row content, and “search hides featured” etc deterministically. Happy for this to land as a follow-up.

louiswol94 · 2026-05-05T19:07:23Z

PlayWright tests are failing

… API

…ptions

…t page

…nto toolbar

…onomy

…mpty results

… header

… current page

…ppets

…dcoded URL

sheabunge · 2026-05-12T07:59:49Z

+		const requestId = nextRequestId()
+		setIsSearching(true)
+
+		api.getResponse<CloudSnippetSchema[]>(


Huge amount of duplicated code here.

sheabunge · 2026-05-12T08:00:30Z

+			return
+		}
+
+		clearTimeout(searchTimerRef.current)


What's the purpose of the timeout? Are we worried about requests to the API taking too long in a way that Axios is unprepared to handle?

sheabunge · 2026-05-12T08:00:51Z

 	const [error, setError] = useState(false)
+	const [isFeatured, setIsFeatured] = useState(false)
+
+	const searchTimerRef = useRef<ReturnType<typeof setTimeout>>()


I do not understand the use of useRef here instead of useState.

sheabunge · 2026-05-12T08:02:14Z

+			'site_host'  => wp_parse_url( get_site_url(), PHP_URL_HOST ),
+		];
+
+		foreach ( [ 'category', 'type', 'status' ] as $key ) {


Is there actually any difference between passing these parameters with empty values versus not passing them at all?

sheabunge · 2026-05-12T08:03:31Z

+		delete_transient( self::CATEGORIES_TRANSIENT_KEY );
+		delete_transient( 'cs_codevault_snippets' );
+
+		$wpdb->query(


I think it'd be much better to avoid a direct DB query here if we can avoid it, especially on a core table. Is there a reason why delete_transient can't do the job for us?

Featured cache keys are dynamic: cs_featured_snippets_p{page}_pp{per_page}_{md5(filters)}. WordPress delete_transient() only removes one exact key, so “reset all featured caches” needs either LIKE delete, a registry of keys in a single transient, or a fixed key with a generation/version bump. The raw query is there because of that key shape; if we want to drop SQL, we need to switch the cache strategy.

louiswol94 · 2026-05-18T19:09:53Z

+		// Wait for the loading spinner to disappear, indicating the featured request completed.
+		await page.locator('.cloud-search .components-spinner').waitFor({ state: 'hidden', timeout: TIMEOUTS.DEFAULT }).catch(() => undefined)
+
+		const featuredHeading = page.locator('.cloud-featured-heading')


Tests target .cloud-featured-heading while the heading uses .cloud-snippets-heading

louiswol94 · 2026-05-18T19:11:06Z

+			'category' => [
+				'description' => esc_html__( 'Filter by category name (comma-separated).', 'code-snippets' ),
+				'type'        => 'string',
+				'default'     => '',
+			],
+			'type'     => [
+				'description' => esc_html__( 'Filter by language/type name (comma-separated).', 'code-snippets' ),
+				'type'        => 'string',
+				'default'     => '',
+			],


get_filter_args() documents category/type as names (and comma-separated strings), while CloudSearchFilters and the UI use numeric IDs.

louiswol94 · 2026-05-18T19:12:34Z

+		$filters = $this->extract_filters( $request );
+		$cloud_snippets = Cloud_API::fetch_search_results( $method, $query, $page, $per_page, $filters );
+
+		$results = [];
+
+		foreach ( $cloud_snippets->snippets as $snippet ) {
+			$results[] = $snippet->get_fields();
+		}
+
+		$response = rest_ensure_response( $results );
+
+		$response->header( 'X-WP-Total', $cloud_snippets->total_snippets );
+		$response->header( 'X-WP-TotalPages', $cloud_snippets->total_pages );
+
+		if ( ! empty( $cloud_snippets->available_filters ) ) {
+			$response->header( 'X-WP-Filters', wp_json_encode( $cloud_snippets->available_filters ) );
+		}


get_items and get_featured_items both map snippets to arrays and set X-WP-Total, X-WP-TotalPages, X-WP-Filters. A private helper would reduce drift when one path is fixed and the other is not.

louiswol94 · 2026-05-18T19:13:38Z

+	 *
+	 * @return array<int, array{id: int, name: string, snippet_count: int}> List of types.
+	 */
+	public static function get_cloud_types(): array {


Cloud_API::get_cloud_types() / get_cloud_categories() and the new routes look unused by the React diff (filters come from headers).

louiswol94 · 2026-05-18T19:14:34Z

+	private function extract_filters( WP_REST_Request $request ): array {
+		return array_filter(
+            [
+				'category' => $request->get_param( 'category' ) ?? '',
+				'type'     => $request->get_param( 'type' ) ?? '',
+				'status'   => $request->get_param( 'status' ) ?? '',
+			]
+        );
+	}


Double check the tab spacings here

code-snippets-bot added 7 commits March 30, 2026 21:37

fix: remove duplicate get_featured_snippets method and constants

5e31860

fix: update featured endpoint URL to /api/v1/public/featured

9021e75

fix: debounce cloud search, prevent stale fetch overwriting results

f50b84f

fix: correct test mock URL match and PHP 7.4 compatibility in feature…

fbdd691

…d tests

louiswol94 approved these changes May 5, 2026

View reviewed changes

louiswol94 added the run-tests Trigger automated tests label May 5, 2026

imantsk added 16 commits May 11, 2026 10:29

Merge branch 'core/core-beta' into core/feat/community-page

9d89072

feat: add pagination params to featured snippets API calls

5c2917b

feat: pass pagination params in featured snippets fetch request

162613f

fix: align tablenav nav to the right on community page

dcdcaa4

feat: add types endpoint proxy and server-side filter params to cloud…

f68bbc9

… API

feat: move filters to server-side and add type dropdown to cloud search

b0c5298

feat: add categories endpoint proxy for server-side category filter o…

f8e3939

…ptions

feat: fetch categories from API for filter dropdown instead of curren…

e00f0bf

…t page

fix: add wp-header-end marker to prevent admin notices from jumping i…

1b107d9

…nto toolbar

feat: move search spinner inside submit button to prevent layout shift

c5f4761

fix: build filter options from search results instead of full API tax…

188344c

…onomy

feat: add consistent cloud snippets heading with filters visible on e…

c4f96cf

…mpty results

feat: pass available_filters from cloud API response via X-WP-Filters…

9e84cda

… header

feat: build filter dropdowns from available_filters header instead of…

11c4e26

… current page

feat: use integer IDs for filter params instead of name strings

3015f50

fix: replace button label with spinner during cloud search loading

ff4f47c

imantsk changed the title ~~feat: Featured Snippets integration for Community page~~ feat: Featured Snippets + server-side filters + type dropdown for Community page May 11, 2026

imantsk changed the title ~~feat: Featured Snippets + server-side filters + type dropdown for Community page~~ feat: Featured Snippets, search and filtering for Community page May 11, 2026

imantsk added 2 commits May 11, 2026 17:50

fix: resolve short ternary lint error and line length warnings

f1396be

fix: update featured snippets PHPUnit tests for new API response format

97b366a

imantsk added 4 commits May 11, 2026 18:11

fix: correct community cloud URL in Playwright test constants

e8e85b7

fix: clear cloud API transients when resetting caches from settings

7b067ed

feat: show orange pro-only button that opens upsell modal for pro sni…

bac2d4b

…ppets

fix: use cloud API base URL for codevault author links instead of har…

acabd43

…dcoded URL

sheabunge reviewed May 12, 2026

View reviewed changes

louiswol94 reviewed May 18, 2026

View reviewed changes

Conversation

code-snippets-bot commented May 5, 2026 • edited by imantsk Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Featured Snippets

Server-Side Filters

New Endpoints Proxied

Cloud Snippet Model

UI

Test plan

Uh oh!

louiswol94 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

louiswol94 commented May 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

code-snippets-bot commented May 5, 2026 •

edited by imantsk

Loading

louiswol94 left a comment •

edited

Loading

louiswol94 commented May 5, 2026 •

edited

Loading