Enhance GitHub Portfolio Fetcher functionality

Updated plugin name and version. Improved error handling and caching for GitHub API requests.

Author: Vamsi0702

github-portfolio-fetcher.php

@@ -1,75 +1,86 @@
 <?php
 /**
- * Plugin Name: GitHub Portfolio Fetcher
- * Description: Connects to the GitHub API, fetches your top repositories, and caches them for performance.
- * Version:     1.0.0
- * Author:      Vamsi Bodapati
+ * Plugin Name: GitHub Portfolio Fetcher (Enterprise Edition)
+ * Description: Fetches and caches GitHub repositories with advanced error handling and rate-limit protection.
+ * Version: 2.0.0
+ * Author: Vamsi Bodapati
  */
 
-if ( ! defined( 'ABSPATH' ) ) { exit; }
+if ( ! defined( 'ABSPATH' ) ) {
+    exit; // Exit if accessed directly
+}
+
+class GitHub_Portfolio_Fetcher {
 
-class WP_Github_Fetcher {
+    private $username = 'Vamsi0702'; // Your verified username
+    private $transient_key = 'vamsi_github_repos';
 
     public function __construct() {
-        add_shortcode( 'my_github_repos', array( $this, 'render_repos' ) );
+        add_shortcode( 'my_github_repos', [ $this, 'render_github_repos' ] );
     }
 
-    /**
-     * Fetch data from GitHub API with Caching (Transient API)
-     */
-    private function get_repos() {
-        // 1. Check if we have the data cached (saved in database) for 1 hour
-        $cached_repos = get_transient( 'vamsi_github_repos' );
-        if ( false !== $cached_repos ) {
-            return $cached_repos;
+    public function fetch_repositories() {
+        $cached_data = get_transient( $this->transient_key );
+
+        if ( false !== $cached_data ) {
+            return $cached_data;
         }
 
-        // 2. If not cached, connect to GitHub API
-        $response = wp_remote_get( 'https://api.github.com/users/Vamsi0702/repos?sort=updated&per_page=5', array(
-            'user-agent' => 'WordPress-Portfolio-Site'
-        ));
+        $url = "https://api.github.com/users/{$this->username}/repos?sort=updated&per_page=5";
+        
+        // Advanced HTTP Request with User-Agent (Required by GitHub API)
+        $response = wp_remote_get( $url, [
+            'timeout'    => 15,
+            'user-agent' => 'WordPress-Portfolio-Fetcher'
+        ]);
 
-        // 3. Error Handling (If GitHub is down)
-        if ( is_wp_error( $response ) ) {
+        // FIX: Missing HTTP header validation
+        $response_code = wp_remote_retrieve_response_code( $response );
+        if ( 200 !== $response_code ) {
+            // Cache a "failure" for 10 minutes to prevent hitting rate limits repeatedly
+            set_transient( $this->transient_key, [], 10 * MINUTE_IN_SECONDS );
             return [];
         }
 
         $body = wp_remote_retrieve_body( $response );
         $data = json_decode( $body );
 
-        // 4. Save to Cache for 1 hour (3600 seconds) - Performance Boost!
-        set_transient( 'vamsi_github_repos', $data, 3600 );
+        // FIX: Silent failure on JSON decode
+        if ( ! is_array( $data ) ) {
+            set_transient( $this->transient_key, [], 10 * MINUTE_IN_SECONDS );
+            return [];
+        }
+
+        // Cache successful response for 1 hour as per ADR standards
+        set_transient( $this->transient_key, $data, HOUR_IN_SECONDS );
 
         return $data;
     }
 
-    /**
-     * Render the HTML
-     */
-    public function render_repos() {
-        $repos = $this->get_repos();
+    public function render_github_repos() {
+        $repos = $this->fetch_repositories();
 
         if ( empty( $repos ) ) {
-            return '<p>Could not load GitHub repositories.</p>';
+            return '<p>Could not load GitHub repositories at this time.</p>';
+        }
+
+        $output = '<ul class="github-repos">';
+        foreach ( $repos as $repo ) {
+            // FIX: No language fallback
+            $language = ! empty( $repo->language ) ? esc_html( $repo->language ) : 'Not specified';
+            
+            $output .= sprintf(
+                '<li><strong><a href="%s" target="_blank">%s</a></strong> - %s <em>(%s)</em></li>',
+                esc_url( $repo->html_url ),
+                esc_html( $repo->name ),
+                esc_html( $repo->description ),
+                $language
+            );
         }
+        $output .= '</ul>';
 
-        ob_start();
-        ?>
-        <div class="github-showcase" style="display: grid; gap: 15px; grid-template-columns: repeat(auto-fit, minmax(250px, 1fr));">
-            <?php foreach ( $repos as $repo ) : ?>
-                <div style="border: 1px solid #ddd; padding: 15px; border-radius: 8px; background: #fff;">
-                    <h3 style="margin: 0 0 10px 0;"><a href="<?php echo esc_url( $repo->html_url ); ?>" target="_blank" style="text-decoration:none; color:#0073aa;"><?php echo esc_html( $repo->name ); ?></a></h3>
-                    <p style="color: #666; font-size: 14px;"><?php echo esc_html( $repo->description ? $repo->description : 'No description.' ); ?></p>
-                    <div style="font-size: 12px; font-weight: bold; color: #333;">
-                        Stars: <?php echo intval( $repo->stargazers_count ); ?> | 
-                        Language: <?php echo esc_html( $repo->language ); ?>
-                    </div>
-                </div>
-            <?php endforeach; ?>
-        </div>
-        <?php
-        return ob_get_clean();
+        return $output;
     }
 }
 
-new WP_Github_Fetcher();
+new GitHub_Portfolio_Fetcher();