add story refresh button

Occasionally enclosure urls become invalid after feed providers delete
the original file. In the case I found, the latest version of the feed
had a valid enclosure url, so refreshing the story from the feed should
work. This is the first time something like this has happened in my
experience, so I think a manual refresh should be sufficient for these
cases. I considered doing an automatic refresh, but wanted to minimize
negative side effects from changing stories after the fact.

Author: mockdeep

.eslint_todo.ts

@@ -1,5 +1,5 @@
 // This configuration was generated by `exe/eslint_autogen`
-// on 2026-07-04 16:59:21 UTC.
+// on 2026-07-06 17:55:24 UTC.
 // The point is for the user to remove these configuration records
 // one by one as the offenses are removed from the code base.
 
@@ -97,7 +97,7 @@ const config: Linter.Config[] = [
       "@stylistic/quote-props": "off",
     },
   },
-  // Offense count: 40
+  // Offense count: 41
   {
     files: [
       "app/javascript/application.ts",
@@ -198,7 +198,7 @@ const config: Linter.Config[] = [
       "@typescript-eslint/no-unsafe-assignment": "off",
     },
   },
-  // Offense count: 177
+  // Offense count: 181
   {
     files: [
       "app/javascript/application.ts",
@@ -210,7 +210,7 @@ const config: Linter.Config[] = [
       "@typescript-eslint/no-unsafe-call": "off",
     },
   },
-  // Offense count: 260
+  // Offense count: 266
   {
     files: [
       "app/javascript/application.ts",
@@ -267,10 +267,12 @@ const config: Linter.Config[] = [
       "@typescript-eslint/strict-boolean-expressions": "off",
     },
   },
-  // Offense count: 17
+  // Offense count: 21
   {
     files: [
       "app/javascript/application.ts",
+      "spec/javascript/controllers/story_refresh_controller_spec.ts",
+      "spec/javascript/helpers/api_spec.ts",
       "spec/javascript/spec/models/story_spec.ts",
       "spec/javascript/spec/views/story_view_spec.ts",
     ],
@@ -326,7 +328,7 @@ const config: Linter.Config[] = [
       "func-style": "off",
     },
   },
-  // Offense count: 9
+  // Offense count: 10
   {
     files: [
       "app/javascript/application.ts",
@@ -510,9 +512,11 @@ const config: Linter.Config[] = [
       "vars-on-top": "off",
     },
   },
-  // Offense count: 3
+  // Offense count: 6
   {
     files: [
+      "spec/javascript/controllers/story_refresh_controller_spec.ts",
+      "spec/javascript/helpers/api_spec.ts",
       "spec/javascript/spec/views/story_view_spec.ts",
     ],
     rules: {

.rubocop_todo.yml

@@ -156,7 +156,7 @@ RSpec/LeakyLocalVariable:
     - 'spec/requests/stories_controller_spec.rb'
     - 'spec/utils/feed_discovery_spec.rb'
 
-# Offense count: 18
+# Offense count: 19
 # Configuration parameters: EnforcedStyle.
 # SupportedStyles: allow, expect
 RSpec/MessageExpectation:
@@ -165,6 +165,7 @@ RSpec/MessageExpectation:
     - 'spec/integration/feed_importing_spec.rb'
     - 'spec/models/migration_status_spec.rb'
     - 'spec/repositories/story_repository_spec.rb'
+    - 'spec/system/stories_index_spec.rb'
     - 'spec/tasks/remove_old_stories_spec.rb'
     - 'spec/utils/i18n_support_spec.rb'
 
@@ -207,12 +208,13 @@ RSpec/NoBeforeHook:
     - 'spec/system/authentication_spec.rb'
     - 'spec/system/profile_spec.rb'
 
-# Offense count: 5
+# Offense count: 6
 # Configuration parameters: IgnoreNameless, IgnoreSymbolicNames.
 RSpec/VerifiedDoubles:
   Exclude:
     - 'spec/commands/feed/fetch_one_spec.rb'
     - 'spec/commands/feed/find_new_stories_spec.rb'
+    - 'spec/commands/story/refresh_from_feed_spec.rb'
     - 'spec/tasks/remove_old_stories_spec.rb'
 
 # Offense count: 1

app/assets/stylesheets/application.css

@@ -258,7 +258,7 @@ li.story.open .story-preview {
   float: right;
 }
 
-.story-keep-unread, .story-starred {
+.story-keep-unread, .story-starred, .story-refresh {
   display: inline-block;
   cursor: pointer;
   -webkit-touch-callout: none;
@@ -379,6 +379,8 @@ p.story-details {
 /* end Wordpress hacks */
 
 .story-actions-container {
+  display: flex;
+  justify-content: space-between;
   border-top: 2px solid #FAF2E5;
   height: 28px;
   line-height: 30px;

app/commands/story/refresh_from_feed.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module RefreshFromFeed
+  class << self
+    def call(story)
+      entry = find_entry(story)
+      return false if entry.nil?
+
+      StoryRepository.update_from_entry(story, entry)
+
+      true
+    rescue StandardError => e
+      Rails.logger.error(
+        "Something went wrong when refreshing story #{story.id}: #{e}"
+      )
+
+      false
+    end
+
+    private
+
+    def find_entry(story)
+      response = SafeFetch.body(story.feed.url)
+      raw_feed = Feedjira.parse(response)
+
+      raw_feed.entries.find { |entry| entry.id == story.entry_id }
+    end
+  end
+end

app/controllers/stories_controller.rb

@@ -22,6 +22,16 @@ def update
     head(:no_content)
   end
 
+  def refresh
+    story = authorization.check(StoryRepository.fetch(params[:id]))
+
+    if RefreshFromFeed.call(story)
+      render(json: story)
+    else
+      head(:unprocessable_content)
+    end
+  end
+
   def mark_all_as_read
     stories = authorization.scope(Story.where(id: params[:story_ids]))
     MarkAllAsRead.call(stories.ids)

app/javascript/application.ts

@@ -130,6 +130,11 @@ var StoryView = Backbone.NativeView.extend({
       this.model.set({is_read: detail.isRead, keep_unread: detail.keepUnread}, {silent: true});
       this.model.trigger('change:is_read');
     });
+    this.el.addEventListener('story-refresh:refreshed', (e) => {
+      this.model.set(e.detail.story, {silent: true});
+      this.render();
+      this.itemOpened();
+    });
   },
 
   itemOpened: function() {
@@ -164,12 +169,13 @@ var StoryView = Backbone.NativeView.extend({
       this.el.classList.add('keepUnread');
     }
     Object.assign(this.el.dataset, {
-      controller: "star-toggle keep-unread-toggle",
+      controller: "star-toggle keep-unread-toggle story-refresh",
       keepUnreadToggleIdValue: String(jsonModel.id),
       keepUnreadToggleIsReadValue: String(jsonModel.is_read),
       keepUnreadToggleKeepUnreadValue: String(jsonModel.keep_unread),
       starToggleIdValue: String(jsonModel.id),
       starToggleStarredValue: String(jsonModel.is_starred),
+      storyRefreshIdValue: String(jsonModel.id),
       unreadCountTarget: "story",
     });
     return this;

app/javascript/controllers/index.ts

@@ -21,5 +21,8 @@ application.register("mark-all-as-read", MarkAllAsReadController);
 import StarToggleController from "./star_toggle_controller";
 application.register("star-toggle", StarToggleController);
 
+import StoryRefreshController from "./story_refresh_controller";
+application.register("story-refresh", StoryRefreshController);
+
 import UnreadCountController from "./unread_count_controller";
 application.register("unread-count", UnreadCountController);

app/javascript/controllers/story_refresh_controller.ts

@@ -0,0 +1,19 @@
+import {Controller} from "@hotwired/stimulus";
+
+import {refreshStory} from "helpers/api";
+
+export default class extends Controller {
+  static override values = {id: String};
+
+  declare idValue: string;
+
+  async refresh(): Promise<void> {
+    const response = await refreshStory(this.idValue);
+    if (!response.ok) {
+      throw new Error(`Failed to refresh story ${this.idValue}`);
+    }
+
+    const story: unknown = await response.json();
+    this.dispatch("refreshed", {bubbles: true, detail: {story}});
+  }
+}

app/javascript/helpers/api.ts

@@ -19,4 +19,11 @@ async function updateStory(
   });
 }
 
-export {updateStory};
+async function refreshStory(id: string): Promise<Response> {
+  return fetch(`/stories/${id}/refresh`, {
+    headers: {"X-CSRF-Token": csrfToken()},
+    method: "POST",
+  });
+}
+
+export {refreshStory, updateStory};

app/repositories/story_repository.rb

@@ -17,6 +17,17 @@ def self.add(entry, feed)
     )
   end
 
+  def self.update_from_entry(story, entry)
+    feed = story.feed
+
+    story.update!(
+      title: extract_title(entry),
+      permalink: extract_url(entry, feed),
+      enclosure_url: safe_normalize_url(entry.try(:enclosure_url), feed.url),
+      body: extract_content(entry)
+    )
+  end
+
   def self.fetch(id)
     Story.find(id)
   end